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.
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
[#2290] TrackingEventProcessor
does not wait for his worker threads to shut down
#2292
Conversation
…ker threads to shut down Signed-off-by: Nils Christian Ehmke <nils-christian.ehmke@bmiag.de>
messaging/src/main/java/org/axonframework/eventhandling/TrackingEventProcessor.java
Outdated
Show resolved
Hide resolved
…ker threads to shut down Signed-off-by: Nils Christian Ehmke <nils-christian.ehmke@bmiag.de>
TrackingEventProcessor
does not wait for his worker threads to shut down
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some small pointers. Otherwise, this looks good to me.
messaging/src/main/java/org/axonframework/eventhandling/TrackingEventProcessor.java
Outdated
Show resolved
Hide resolved
messaging/src/main/java/org/axonframework/eventhandling/TrackingEventProcessor.java
Outdated
Show resolved
Hide resolved
messaging/src/main/java/org/axonframework/eventhandling/TrackingEventProcessor.java
Outdated
Show resolved
Hide resolved
…ngEventProcessor.java Co-authored-by: Steven van Beelen <steven.vanbeelen@axoniq.io>
messaging/src/main/java/org/axonframework/eventhandling/TrackingEventProcessor.java
Outdated
Show resolved
Hide resolved
messaging/src/main/java/org/axonframework/eventhandling/TrackingEventProcessor.java
Outdated
Show resolved
Hide resolved
Provided the commit suggestions just now, @nils-christian. Might be fair if we provide a style guide for Eclipse somewhere in the future too 😅 |
…ngEventProcessor.java Co-authored-by: Steven van Beelen <steven.vanbeelen@axoniq.io>
…ngEventProcessor.java Co-authored-by: Steven van Beelen <steven.vanbeelen@axoniq.io>
Suggestions applied. Thank you @smcvb. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My concerns have been addressed, hence I'm approving this pull request.
TrackingEventProcessor does not wait for his worker threads to shut down
Signed-off-by: Nils Christian Ehmke nils-christian.ehmke@bmiag.de
This pull request resolves #2290.