search

open-telemetry / opentelemetry-collector-contrib

Contrib repository for the OpenTelemetry Collector
https://opentelemetry.io
Apache License 2.0
3.07k stars 2.37k forks source link

Replace off-spec statsdreceiver Summary usage with new exponential histogram #5742

Closed jmacd closed 2 years ago

jmacd commented 3 years ago

Is your feature request related to a problem? Please describe.

Also mentioned in #5667, the Summary point issued by statsdreceiver uses delta temporality. This is not specified, and delta temporality will never be specified for Summary points. A histogram or exponential histogram is the appropriate choice.

Describe the solution you'd like Use the exponential histogram aggregator built for this purpose as well as for the OTel-Go SDK, drafted here: https://github.com/open-telemetry/opentelemetry-go-contrib/pull/1353

Describe alternatives you've considered Giving Summary an aggregation temporality or letting this kind of use-case stand as a "degenerate cumulative" temporality have both been considered.

Additional context This requires the exponential histogram to be introduced to the OTel collector first: https://github.com/open-telemetry/opentelemetry-collector/issues/4197

jmacd commented 3 years ago

The "off spec" remark is backed up here: https://github.com/open-telemetry/opentelemetry-specification/pull/2002