opentelemetry-java icon indicating copy to clipboard operation
opentelemetry-java copied to clipboard

Published 20 hours ago • verified publisher icon open-telemetry

Update exponential histogram to use upper-inclusive boundaries

Open jmacd opened this issue 3 years ago • 2 comments

Describe the request

Specification PR https://github.com/open-telemetry/opentelemetry-specification/pull/2633 has changed the boundary conditions for exponential histograms and specified that SDKs SHOULD be exact for powers-of-two. This implies a fairly complex change, see https://github.com/open-telemetry/opentelemetry-go/pull/2982 for example.

jmacd avatar Aug 18 '22 16:08 jmacd

I've been working on this! Will have a PR sometime today.

jack-berg avatar Aug 18 '22 16:08 jack-berg

FYI, the corresponding change to the proto definition has yet to be merged / released. Despite this, I think its ok for us to proceed with this change beforehand given the experimental status of our exponential histograms.

jack-berg avatar Aug 18 '22 19:08 jack-berg