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

tracing: support a sample rate for tracers #45891

Open
ajwerner opened this issue Mar 9, 2020 · 2 comments
Open

tracing: support a sample rate for tracers #45891

ajwerner opened this issue Mar 9, 2020 · 2 comments

Comments

@ajwerner
Copy link
Collaborator

@ajwerner ajwerner commented Mar 9, 2020

Is your feature request related to a problem? Please describe.

Currently cockroach supports external tracing services using the zipkin or lightstep protocols. When these features are enabled, 100% of all requests are traced. This leads to serious overhead both inside of Cockroach and in the trace collection infrastructure. Tracing can provide value even at very low sampling rates. Furthermore, with new features to enable tracing on-demand for certain statement fingerprints, users could gain additional value out of an existing trace collection infrastructure.

Describe the solution you'd like

Add a cluster setting to control the sampling rate for trace collection.

Additional context

See this discussion for the original conversation #45578 (comment).

@TylerJGabb
Copy link

@TylerJGabb TylerJGabb commented Jul 1, 2020

I'd like to start working on this. Currently reading the wiki and trying to figure out how to start, and how to be assigned.

@ajwerner
Copy link
Collaborator Author

@ajwerner ajwerner commented Jul 1, 2020

Just this comment is good enough. A pull request would be great. In general we haven't had actual contention with community contributions competing to fix the same issue. I don't anticipate that here. If somebody internally were to be working on it, it would get assigned.

If you run in to any trouble or want any pointers on how to approach it, let me know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Observability
  
Awaiting triage
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.