allthingssecurity commented on PR #28751: URL: https://github.com/apache/camel/pull/28751#issuecomment-6540601115
@davsclaus thanks for the review. In b5f0151: 1. Upgrade guide: a bullet for the redelivery case (`inheritErrorHandler=true` or the attempt route's own error handler, last redelivery failing with a different exception), with your `ConnectException` / `HttpOperationFailedException` example. 2. `ThrottlingExceptionRoutePolicy`: the follow-up JIRA is drafted. It will be filed and linked from CAMEL-26328 before this is merged. 3. The guide and the PR description now say that the `FatalFallbackErrorHandler` case is not a 4.23 regression: it changes behaviour that dates back to 2.16, on purpose. 4. The section now sits with the other camel-core entries, before the camel-util ones. 5. `FailOverLoadBalanceSuppressedExceptionTest`: a comment says that there is one suppressed exception because of `RedeliveryErrorHandler.isSame`. `FailOverLoadBalanceSuppressedExceptionTest` and the load balancer / failover tests pass (87 tests). _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]
