Skip to content

Conversation

bertschneider
Copy link
Contributor

Unsubscribing message handlers while processing an in-flight message may lead to a MessageDeliveryException.

Fixes: GH-10362

Copy link
Member

@artembilan artembilan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, run ./gradlew :spring-integration-jdbc:check locally before pushing.

It also would be great if you bring some test to cover this logic.

Thanks

@bertschneider bertschneider force-pushed the GH-10362 branch 2 times, most recently from be9d911 to 33dad8d Compare August 29, 2025 19:01
@artembilan
Copy link
Member

Thank you for an update!
Commenting from the phone.
Will review properly on Tuesday after holidays .

…ribe

Unsubscribing message handlers while processing an in-flight message may lead to
a MessageDeliveryException.

Fixes: spring-projectsGH-10362

Signed-off-by: Norbert Schneider <[email protected]>
Copy link
Member

@artembilan artembilan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.
Will merge when the PR build is green.

@bertschneider
Copy link
Contributor Author

Great! Thanks for your time and support with this PR.

@artembilan artembilan merged commit 1da83bd into spring-projects:main Sep 3, 2025
3 checks passed
@artembilan
Copy link
Member

Merged and cherry-picked to 6.5.x & 6.4.x.

Thank you for contribution; looking forward for more!

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.

Unsubscribe of message handler while processing PostgresSubscribableChannel.notifyUpdate()
2 participants