Croway opened a new pull request, #23179: URL: https://github.com/apache/camel/pull/23179
## Summary - Skip setting `preparingShutdown = true` in `prepareShutdown()` when `suspendOnly = true`, preventing silent transaction rollback and redelivery suppression after `context.suspend()` / `context.resume()` - The flag is meant to signal a permanent shutdown, not a temporary suspend — the `suspendOnly` parameter already distinguishes these cases but was never checked - Fixes `RedeliveryErrorHandler`, `camel-spring` `TransactionErrorHandler`, and `camel-jta` `TransactionErrorHandler` ## Test plan - [x] `RedeliveryErrorHandlerSuspendResumeTest` — verifies redelivery works after context and route suspend/resume (with `allowRedeliveryWhileStopping(false)`) - [x] `TransactionErrorHandlerSuspendResumeTest` (camel-jta) — verifies JTA transactions commit after context and route suspend/resume - [x] `TransactionalClientDataSourceSuspendResumeTest` (camel-spring-xml) — verifies Spring transactions commit after context and route suspend/resume - [x] Existing shutdown tests pass (`TransactionalClientDataSourceForcedShutdownTest`, `TransactionErrorHandlerShutdownTest`, `NotAllowRedeliveryWhileStoppingTest`, `RedeliveryErrorHandlerNoRedeliveryOnShutdownTest`) _Claude Code on behalf of Federico Mariani_ -- 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]
