opentelemetry-java icon indicating copy to clipboard operation
opentelemetry-java copied to clipboard

Invoke async metric callbacks in the context class loader from the time when the instrument was registered

Open trask opened this issue 10 months ago • 3 comments

See discussion in https://github.com/open-telemetry/opentelemetry-java-instrumentation/pull/9000

trask avatar Aug 08 '23 20:08 trask

I feel like we talked about this before and concluded that the current behavior is correct 🤔. Is this related to https://github.com/open-telemetry/opentelemetry-java-instrumentation/issues/7220?

jack-berg avatar Aug 10 '23 15:08 jack-berg

I feel like we talked about this before and concluded that the current behavior is correct 🤔

I'm not remembering, maybe we discussed it in the context of the micrometer bridge implementation, @mateuszrzeszutek do you recall?

Is this related to open-telemetry/opentelemetry-java-instrumentation#7220?

I think so, though I sent #9000 based on a different report that we got against our distro (https://github.com/open-telemetry/opentelemetry-java-instrumentation/pull/9000#issuecomment-1661725817)

trask avatar Aug 14 '23 17:08 trask

I'm not remembering, maybe we discussed it in the context of the micrometer bridge implementation, @mateuszrzeszutek do you recall?

Unfortunately not, I think we might have talked about it but totally can't recall the contents of our conversation 🙈

While this issue manifested in the micrometer bridge, I believe you can as easily reproduce it using just the OTel API.

mateuszrzeszutek avatar Aug 16 '23 10:08 mateuszrzeszutek