search

beam-telemetry / telemetry_metrics_statsd

Telemetry.Metrics reporter for StatsD-compatible metric servers
https://hexdocs.pm/telemetry_metrics_statsd
MIT License
75 stars 44 forks source link

Change the reported DataDog metric types #31

Closed arkgil closed 4 years ago

arkgil commented 4 years ago

This PR changes how the summary and distribution metrics map to DataDog metrics.

tl;dr is that summary is now reported as DataDog histogram, and distribution as DataDog distribution.

The rationale here is that DataDog histogram is a metric which calculates statistics around measurements, exactly how Telemetry.Metrics summary is defined.

With the removed requirement for distribution buckets in Telemetry.Metrics, we can now always report distribution as DataDog distribution without users having to define buckets for no reason.

arkgil commented 4 years ago

@hkrutzer this might be of interest to you.