Skip to content

Mark the session as not running on any Exception - #2648

Open
otbutz wants to merge 3 commits into
apache:mainfrom
otbutz:session_start_exception
Open

otbutz wants to merge 3 commits into
apache:mainfrom
otbutz:session_start_exception

Conversation

@otbutz

@otbutz otbutz commented Oct 12, 2026 •

Copy link
Copy Markdown

Part of #2647

@rzo1

rzo1 commented Oct 14, 2026

Copy link
Copy Markdown

The widened catches help, but they aren't enough on their own: once the session start is rejected, the reconnect in ActiveMQEndpointWorker.connect() is rejected by the same saturated work manager, and the endpoint gives up for good. I posted a test that reproduces this, RejectedWorkRecoveryTest, in #2647 (comment). It might be worth adding it to this PR together with a retry in connect().

@otbutz
otbutz force-pushed the session_start_exception branch from 2a34adf to 16aa671 Compare October 14, 2026 15:49
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.

2 participants