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

Default bucket boundaries for explicit bucket histogram are different from specification

Open jack-berg opened this issue 2 years ago • 1 comments

The java default explicit bucket histogram bucket boundaries are:

[5, 10, 25, 50, 75, 100, 250, 500, 750, 1_000, 2_500, 5_000, 7_500, 10_000]

But according to the spec should be:

[ 0, 5, 10, 25, 50, 75, 100, 250, 500, 1_000 ]

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

Reopening this since we reverted the fix in #4736.

Need to wait for a resolution on spec PR #2770.

jack-berg avatar Sep 13 '22 21:09 jack-berg

The spec has changed the default boundaries to [ 0, 5, 10, 25, 50, 75, 100, 250, 500, 750, 1000, 2500, 5000, 7500, 10000 ].

Note that the only difference between the new defaults and our current defaults is that the spec has a 0 bucket. We should be able to add this bucket without issue because negative values currently aren't allowed so adding it should be functionally equivalent to what we currently do.

jack-berg avatar Sep 21 '22 17:09 jack-berg