-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
[chore] bump go-grpc-compression #7863
[chore] bump go-grpc-compression #7863
Conversation
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #7863 +/- ##
=======================================
Coverage 91.14% 91.14%
=======================================
Files 298 298
Lines 14915 14915
=======================================
Hits 13594 13594
Misses 1045 1045
Partials 276 276 ☔ View full report in Codecov by Sentry. |
@codeboten need a rebase |
@bogdandrutu needs more than a rebase. The dependency on otelgrpc instrumentation causes an ambiguous import, which I don't think we can address until the instrumentation is updated. |
@codeboten you can add a |
@bogdandrutu but this replace would need to be in the builder configuration for anyone building the collector w/ the otlp exporter right? |
Until the thing is fixed upstream we can hardcode in the builder as one of the replacements always? |
This helps address an issue where zstd compression causes significant memory pressure. Fixes open-telemetry#7862 Signed-off-by: Alex Boten <[email protected]>
Signed-off-by: Alex Boten <[email protected]>
0c61adf
to
75ffc9f
Compare
This address an issue where zstd compression causes significant memory pressure. Fixes #7862