-
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
[processor/metricstransformprocessor] fix copy exponential histogram #27410
[processor/metricstransformprocessor] fix copy exponential histogram #27410
Conversation
Thanks for the fix, @mfranklin! |
@mfranklin can you please add a changelog item to https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/.chloggen? |
@mfranklin: You can run |
891575f
to
f5c5986
Compare
Changelog added. |
…pen-telemetry#27410) Fixes a bug where the wrong function was called for the exponential histogram type during the copy operation. Fixes open-telemetry#27409 Added a test for `combine` that covers the failing code path.
Description:
Fixes a bug where the wrong function was called for the exponential histogram type during the copy operation.
Link to tracking Issue:
Fixes #27409
Testing:
Added a test for
combine
that covers the failing code path.Documentation:
No documentation added as this is a simple bug fix.