Add zstd compression on exporter/aws3#44542
Conversation
|
|
|
Welcome, contributor! Thank you for your contribution to opentelemetry-collector-contrib. Important reminders:
A maintainer will review your pull request soon. Thank you for helping make OpenTelemetry better! |
atoulme
left a comment
There was a problem hiding this comment.
this works ; note some other work is to derive the compression from the content-type of the response headers. This is complementary to this approach.
|
Thank you for your contribution @ovaladares! 🎉 We would like to hear from you about your experience contributing to OpenTelemetry by taking a few minutes to fill out this survey. If you are getting started contributing, you can also join the CNCF Slack channel #opentelemetry-new-contributors to ask for guidance and get help. |
Description
Add support for
zstdcompression onexporter/awss3Testing
Automated tests passed.
Manual validation, downloaded files from S3 and decompressed using
zstdCLI.Documentation
Updated the existent documentation to add
zstdoption as compression.