allthingssecurity commented on PR #29127: URL: https://github.com/apache/camel/pull/29127#issuecomment-6625271627
Thanks @davsclaus, all three done in e6e661ed6371: 1. `CamelInternalProcessor` now catches a `Throwable` and `AsyncAfterTask.takeOver` takes a `Throwable`, as `SimpleTask`, `RedeliveryTask` and `NoErrorHandler`; new `doTryWithDoFinallyProcessorThrowsError` throws a `NoClassDefFoundError` and checks that the exchange fails with it and the `doFinally` runs. 2. The tests that depend on reused tasks (the late callback, the callback called twice, the next exchange, and the race) are in `StepWithoutErrorHandlerProcessorThrowsTestSupport`; the main test and the pooled test both extend it, so the pooled test runs only those 5, as in the sibling fixes. The race test sends 50 exchanges. 3. Wording: the description no longer says "when ... are merged", the multicast output with a redelivering error handler is described as redelivered on main since CAMEL-26446 (also the comment in `multicastOutputRedelivered`), the upgrade-guide clause is gone, and the `DefaultProducerCache` sentence with the placeholder key is dropped. Local run: 21 + 5 + 16 (`NoErrorHandlerProcessorThrowsTest`) and the other error-handler, `doTry` and multicast tests, 211 in all, 0 failures. _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]
