Skip to content
This repository has been archived by the owner on May 23, 2024. It is now read-only.

Support delayed sampling #449

Closed
yurishkuro opened this issue Oct 15, 2019 · 1 comment · Fixed by #466
Closed

Support delayed sampling #449

yurishkuro opened this issue Oct 15, 2019 · 1 comment · Fixed by #466

Comments

@yurishkuro
Copy link
Member

yurishkuro commented Oct 15, 2019 •
edited
Loading

Per meta-ticket jaegertracing/jaeger#1861

Documentation: https://github.com/jaegertracing/jaeger-client-go#delayed-sampling
@vprithvi vprithvi mentioned this issue Oct 15, 2019
Merged
This was referenced Oct 15, 2019
Merged
Merged
Merged
Closed
Merged
Merged
Merged
Merged
@yurishkuro
Copy link
Member Author

yurishkuro commented Oct 23, 2019 •
edited
Loading

Open question: can AdaptiveSampler be configured to return retryable=false from OnCreateSpan()? This would reduce performance penalty for applications that do not use late-binding of span's operation name (i.e. they don't call SetOperationName() after span

Update: fixed by #459

This was referenced Oct 28, 2019
Merged
Merged
Merged
@yurishkuro yurishkuro mentioned this issue Nov 6, 2019
Merged
@yurishkuro yurishkuro mentioned this issue Jan 24, 2020
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Prepare release 2.20 yurishkuro/jaeger-client-go
1 participant
@yurishkuro