allthingssecurity commented on PR #29041: URL: https://github.com/apache/camel/pull/29041#issuecomment-6598569679
Thanks, done in 9d2a548d9048, after merging main in (no conflicts). 1. `NoErrorHandler.process` catches `Throwable`, keeps `throw e` when the callback was already called (a precise rethrow), and uses `takeOver(String exchangeId, Throwable cause)`. New test `processorThrowsError` throws `NoClassDefFoundError`. With the previous `NoErrorHandler` it times out after 10 seconds, and the other 15 tests pass. I updated the PR description. 2. `NoErrorHandlerProcessorThrowsTest` extends `SimpleTaskProcessorThrowsTestSupport` and overrides `start(Processor)` to add `errorHandler(noErrorHandler())`. It inherits the helpers and the late callback and double callback tests, and the copies are gone. The stress test sends 50 exchanges. 3. The takeover WARN includes the exchange id, read in the catch while the exchange is still ours. The class, `SimpleTaskProcessorThrowsTest` and its pooled class ran 20 times in one JVM: 640 tests, 0 failures. `processor.errorhandler.**` plus the error handler, redelivery and dead letter tests: 247, 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]
