Skip to content
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

enhancement: add a new DogStatsD-specific exporter #548

Merged
merged 13 commits into from
Jan 6, 2025
Merged

Conversation

tobz
Copy link
Member

@tobz tobz commented Dec 29, 2024

Context

This PR adds a brand new exporter, metrics-exporter-dogstatsd, which is specifically designed for sending metrics using the DogStatsD protocol, typically to a Datadog Agent or DogStatsD server.

While metrics-exporter-statsd exists and works well for many use cases, it depends on cadence which ultimately limits the maximum performance and capabilities of any exporter depending on it. This shows up in terms of cached access to metrics and things like optimal payload building, and so on.

Further, while libraries like cadence do strive to support DogStatsD-specific features, it doesn't aspire to (to my knowledge) match support with the official DogStatsD client libraries, while metrics-exporter-dogstatsd does. This includes support for client-side aggregation, histogram sampling, smart reporting, and more.

Notes

We've done some unrelated refactoring/cleanup in metrics-util to add the necessary bits for the new exporter.

@tobz tobz added C-exporter Component: exporters such as Prometheus, TCP, etc. E-intermediate Effort: intermediate. T-enhancement Type: enhancement. labels Dec 29, 2024
@tobz tobz merged commit c15fed2 into main Jan 6, 2025
13 checks passed
@tobz tobz deleted the tobz/statsd-exporter branch January 6, 2025 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-exporter Component: exporters such as Prometheus, TCP, etc. E-intermediate Effort: intermediate. T-enhancement Type: enhancement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant