Skip to content

Conversation

gsmet
Copy link
Member

@gsmet gsmet commented Aug 8, 2025

And bubble up the errors to HTTP response, if sync.

And bubble up the errors to HTTP response, if sync.
@gsmet gsmet requested a review from a team as a code owner August 8, 2025 17:22
@gsmet
Copy link
Member Author

gsmet commented Aug 8, 2025

There's one major drawback to the sync approach though: if you have several methods listening to the same event, an error will cancel the rest of the execution, which might not be practical.

That was one of the reasons why I went with the async approach.

I would have to use another approach than CDI observers for the sync case if we want to collect all errors and not stop the execution randomly.

@gsmet
Copy link
Member Author

gsmet commented Sep 29, 2025

We won't pursue this.

@gsmet gsmet closed this Sep 29, 2025
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