-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[exporter/prometheusremotewrite] fix snappy double alloc #34274
[exporter/prometheusremotewrite] fix snappy double alloc #34274
Conversation
Co-authored-by: Raphael Philipe Mendes da Silva <rapphil@gmail.com>
There are a few approval steps that are failing, can you take a look to make sure that they are not related to this change or require merging from mainline? Otherwise this change looks good to me. |
Co-authored-by: Matthew Wear <matthew.wear@gmail.com>
@ben-childs-docusign , It is possible that it is a flaky test. can you take a look into other PRs that were created recently and check if the tests are failing there as well? Is your branch updated with regards to main? As next steps, I think @mwear can add the |
Description:
Snappy Encode function is not using the buffer passed in by prometheus remote write as it is too small. Removing this unused buffer allocation and just passing nil. Snappy will allocate the required buffers.
Link to tracking Issue:
Fixes #34273
Testing:
Ran existing unit tests which cover this code
Ran this in our integration environment and verified reduced allocations.
Before:
After:
Documentation:
Updated changelog