Skip to content

Report errors on writer close.#1341

Open
gemyago wants to merge 1 commit intosegmentio:mainfrom
gemyago:feature/close-errors
Open

Report errors on writer close.#1341
gemyago wants to merge 1 commit intosegmentio:mainfrom
gemyago:feature/close-errors

Conversation

@gemyago
Copy link

@gemyago gemyago commented Oct 18, 2024

Currently, errors (at least in async mode) will only go to logs (if configured). With this change, the Close function will return an error that will fail the caller side in a "loud" way. This helps to identify the issue sooner.

Return error from Close if errors occurred on writer
@seg-atlantis-prod
Copy link

Atlantis commands can't be run on fork pull requests. To enable, set --allow-fork-prs or, to disable this message, set --silence-fork-pr-errors

@seg-atlantis-prod
Copy link

Error parsing command: EOF found when expecting closing quote

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant