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
[#2639] Handler Interceptor support for Dead Letter Processing #2661
Conversation
072f255
to
7f08372
Compare
Kudos, SonarCloud Quality Gate passed! |
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.
I think we need to move the interceptor invocation from the DeadLetteringEventHandlerInvoker
to the DeadLetteredEventProcessingTask
.
In the invoker, we deal with regular event handling, while the processing task is when we handle a dead letter.
Thus, the former is already covered by MessageHandlerInterceptors
, through the Event Processor the invoker's a part of.
3cbeeac
to
5a506bc
Compare
5a506bc
to
4416c44
Compare
The pr was adjusted and can be reviewed again. |
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.
A couple of minor pointers. However, nothing to block over if you ask me. Good job!
...rc/main/java/org/axonframework/eventhandling/deadletter/DeadLetteredEventProcessingTask.java
Outdated
Show resolved
Hide resolved
.../test/java/org/axonframework/eventhandling/deadletter/DeadLetteringEventIntegrationTest.java
Outdated
Show resolved
Hide resolved
...est/java/org/axonframework/eventhandling/deadletter/DeadLetteredEventProcessingTaskTest.java
Show resolved
Hide resolved
.../test/java/org/axonframework/eventhandling/deadletter/DeadLetteringEventIntegrationTest.java
Show resolved
Hide resolved
…tter/DeadLetteringEventIntegrationTest.java Co-authored-by: Steven van Beelen <steven.vanbeelen@axoniq.io>
…o the unit of work.
…ptor was called at all.
Kudos, SonarCloud Quality Gate passed! |
Handler Interceptor support for Dead Letter Processing, using the same interceptors as the processing group.
Fixes #2639