zio / zio-telemetry

ZIO-powered OpenTelemetry library
https://zio.dev/zio-telemetry
Apache License 2.0
113 stars 55 forks source link

OTEL histogram contains two infs #870

Closed andrzejressel closed 3 months ago

andrzejressel commented 3 months ago

When using boundaries fromChunk, Double.MaxValue is added to the end. But OTEL already adds inf. This causes two inf - one is native "inf" and second is just boundary with Double.MaxValue.

Test works because I've used default constructor. It should be changed to fromChunk factory instead.

While histogram can be workarounded by using default Histogram constructor, timer cannot be fixed.