open-telemetry
OTLP exporters should load Compressor using the class loader configured using `setServiceClassLoader`
I guess we could consider dropping the separate class loader for extensions. @trask wdyt?
if we can solve with #7150 that sounds ideal 👍
There is the same issue with https://github.com/open-telemetry/opentelemetry-java/blob/8efade69a16f86164d52a9b9699620fc254ce772/exporters/common/src/main/java/io/opentelemetry/exporter/internal/compression/CompressorUtil.java#L51 An alternative would be to use context class loader. Currently we don't set the context loader in the agent, but we used to. Can't recall why we had to stop setting it.
Originally posted by @laurit in https://github.com/open-telemetry/opentelemetry-java/discussions/7164#discussioncomment-12417409
@jack-berg In your proposal you suggested bringing SpiHelper to opentelemetry-sdk-common. I really liked this idea. Would it be okay for me to proceed with making this change and use it in CompressionUtil for loading the Compressor?
This issue is resolved. Can we close this?
Do you know what PR resolved it? It would be good to reference it when closing, for future reference.
This issue is resolved. Can we close this? @jkwatson @jack-berg
Do you know what PR resolved it? It would be good to reference it when closing, for future reference.
@jkwatson This was resolved in this PR.