allthingssecurity commented on PR #28205: URL: https://github.com/apache/camel/pull/28205#issuecomment-6310217570
Thanks Claus. 1. Pooled exchange: done in fb5ce934f823. The transacted consumer creates the exchange with `createExchange(false)` and releases it with `releaseExchange(exchange, false)` after the commit or rollback (in a `finally`, so also after an empty poll). The synchronous branch is now chosen on `transacted`, so an exchange the consumer releases itself is never routed asynchronously. New `HazelcastSedaTransactedPooledExchangeTest` runs the two tests with `PooledExchangeFactory`; without the change the failed message is committed and never taken again (`Expected: <1> but was: <0>`). The non-transacted path still reads the exception in the callback after the reset; that is the same on main and I'll open a follow-up for it. 2. The member join over 127.0.0.1: no change here; I'll watch it in the CI runs of this PR. _Claude Code on behalf of allthingssecurity_ -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
