Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Persistence] Query does not report failure correctly #5834

Closed
Arkatufus opened this issue Apr 8, 2022 · 1 comment · Fixed by #5835
Closed

[Persistence] Query does not report failure correctly #5834

Arkatufus opened this issue Apr 8, 2022 · 1 comment · Fixed by #5835
Labels
discussion

Comments

@Arkatufus
Copy link
Contributor

Related to #5824

Failure in AllEvents query is reported on debug level, which is incorrect. It seems like error and failure handling in Akka.Persistence.Query is not standardized. We need to standardize error and failure reporting in Akka.Persistence.Query.
@Arkatufus
Copy link
Contributor Author

This is actually not a bug, the original issue stemmed from the fact that the Task was not await-ed, making it a detached Task. The exception thrown by the stream supervisor was not caught.

@Arkatufus Arkatufus mentioned this issue Apr 8, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
discussion
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Change the failure log inside AllEventPublisher from Debug to Error Arkatufus/akka.net
1 participant
@Arkatufus