lmolkova commented on PR #8713: URL: https://github.com/apache/camel/pull/8713#issuecomment-1353840089
@bvahdat sure, I rebased! I believe (again) that I fixed tests. There were two issues: 1. We still need to clean the context in `SendProcessor`: since we have multiple nested spans, we need to clean scope after them. I *think* I found all the new event-related tests. 2. Test was not correct: sometimes aggregation can be scheduled on the same thread on which outer span is still active - it's ok and if we had OTel executor instrumentation enabled, it'd populate this context on other threads too. Added check and description to the test. This behavior was different on Java 11 and Java 17 and affected Java 11 much more. It's stable now on both versions. -- 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: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
