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

Support snappy compressor in CompressorUtil

Open qixiaogang opened this issue 1 year ago • 3 comments

Is your feature request related to a problem? Please describe.

gzip is the only supported default compression for exporter, however gzip consumes much more CPU than snappy, so it will be great to also provide snappy as supported compression.

Describe the solution you'd like

calling setCompression("snappy") for exporter will use snappy compressor. TelemetryExporter<T> exporter = exporterBuilder().setEndpoint(endpoint).setCompression("snappy").build();

Describe alternatives you've considered

No response

Additional context

No response

qixiaogang avatar Mar 11 '24 09:03 qixiaogang

@open-telemetry/java-maintainers could you move this issue to https://github.com/open-telemetry/opentelemetry-java

laurit avatar Mar 11 '24 09:03 laurit

moving to https://github.com/open-telemetry/opentelemetry-java-contrib

@qixiaogang if you are interested, we would probably accept this as a new pluggable compressor, similar to https://github.com/open-telemetry/opentelemetry-java-contrib/tree/main/compressors/compressor-zstd

cc @jack-berg who is the component owner for the zstd contrib compressor

trask avatar Mar 11 '24 23:03 trask

@trask thank you, it is what I want except it is zstd.

qixiaogang avatar Mar 12 '24 10:03 qixiaogang