allthingssecurity commented on PR #28806: URL: https://github.com/apache/camel/pull/28806#issuecomment-6571435108
Thanks for the review. Blocking `process(Exchange)`: done in 24cdd5f72658. The span is no longer ended in a `finally`; when the step throws (or set an exception on the exchange), a failure to end the span is logged at WARN and the step's exception is kept, as on the asynchronous path. The success path still throws the end failure. `ProcessorSpanEndFailureTest` has a third test for it, which fails with the previous strategy. The class ran 20 times in one JVM (60 tests) and the camel-telemetry module tests (183) pass. The PR description is updated. _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]
