-
Notifications
You must be signed in to change notification settings - Fork 77
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
add compression when posting metrics to dd #358
Conversation
c825c1b
to
ef66d0e
Compare
I think adding a configuration option for compression maybe the best way to go here, will rework the PR to support that. |
232a7cd
to
a43e4d3
Compare
a43e4d3
to
1a88201
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Just one minor suggestion.
I was also thinking about adding a minimum size for compression, like 1kb or 2kb, let me know if you think that would be preferable here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, Rob!
Adds gzip compression when posting metrics to Datadog.