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

Add an API to send multiple samples at once #292

Closed

Conversation

iksaif
Copy link
Contributor

@iksaif iksaif commented Dec 6, 2023

DistributionSamples is similar to Distribution, but it lets the client deals with the sampling, rate is passed to the agent and not used for further sampling. WithMaxBufferedMetricsPerContext is ignored when using this method.

Because this is a shift compared to how other methods are behaving, this method is provided only in ClientDirect which provides direct access to some low level dogstatsd protocol features like /~https://github.com/DataDog/datadog-go#extended-aggregation

This is recommended in high performance cases were the overhead of the statsd library might be significant and the sampling is already done by the client.

@iksaif iksaif marked this pull request as ready for review December 14, 2023 12:38
@iksaif iksaif requested a review from a team as a code owner December 14, 2023 12:38
@iksaif iksaif closed this Dec 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants