Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support snappy compressor in CompressorUtil #1222

Open
qixiaogang opened this issue Mar 11, 2024 · 3 comments
Open

Support snappy compressor in CompressorUtil #1222

qixiaogang opened this issue Mar 11, 2024 · 3 comments

Comments

@qixiaogang
Copy link

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

@laurit
Copy link
Contributor

laurit commented Mar 11, 2024

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

@trask
Copy link
Member

trask commented Mar 11, 2024

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 trask transferred this issue from open-telemetry/opentelemetry-java-instrumentation Mar 11, 2024
@qixiaogang
Copy link
Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants