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
Enable priority sampling by default #774
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd say only to update the doc to not show how to disable Priority Sampling. At the moment there is no reason to disable it so let's discard that part.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great, thank you very much!
For reference, we're targeting automatically endpoint v0.4
with a fallback system that downgrades the connection to endpoint v0.3
if missing (404
). This change in the default is reasonably safe.
Also, our Agent added the Priority Sampling endpoint with this PR: https://github.com/DataDog/datadog-trace-agent/pull/310/files#diff-e8961f81006d547c70ab424318805bbaR99
that landed on this release (3 Nov 2017): https://github.com/DataDog/datadog-trace-agent/releases/tag/5.19.0
We want to enable priority sampling by default now.