allthingssecurity commented on PR #28205: URL: https://github.com/apache/camel/pull/28205#issuecomment-6314806362
Thanks Andrea. 1. Checked in Hazelcast 5.4.0: `newTransactionContext()` uses `TransactionOptions.getDefault()` (2 minutes) and `commit()` checks the timeout first. The option description, the generated JSON and DSL, and the upgrade note now say that the exchange must complete within that timeout, or the commit fails, the transaction is rolled back and the message is consumed again. Passing a failed commit to the exception handler goes to a follow-up JIRA. 2. The tests stop the context before they shut the members down (also in `testCompletedExchangesAreCommitted`), and the new classes and methods are package-private (2028f9737fdd). _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]
