-
Notifications
You must be signed in to change notification settings - Fork 981
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DatadogMeterRegistry: add support for publishing distributions #3243
Comments
Thanks for letting us know an HTTP API is available for sending distributions now. We previously abandoned efforts to support this with the Datadog registry due to the lack of API. I do still have concerns about how we could/should best handle such a feature. See my previous comment on it #1056 (comment). Unlike the statsd registry, the datadog registry is a step meter registry, which pushes metrics at step intervals. However, to send the raw data that datadog expects for distributions, we would need to either separately publish distribution data in near-realtime with recording or buffer the raw data for publishing. This would add significant complexity. It may still be worth adding if it provides enough value for users. |
Tbh I personally prefer using |
I realize we already had an open issue for this: #773. I'll close this as a duplicate of the older issue to keep things in one place. |
Duplicate of #773 |
Please describe the feature request.
At the moment
DatadogMeterRegistry
doesn't support publishing percentile histogram, while DD's API seems to support publishing distributions already: https://docs.datadoghq.com/api/latest/metrics/#submit-distribution-points .Rationale
For
DatadogMeterRegistry
to reach feature parity with the statsd registryThe text was updated successfully, but these errors were encountered: