-
Notifications
You must be signed in to change notification settings - Fork 163
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
Probability sampler composition rules #175
Probability sampler composition rules #175
Conversation
… unknown in the span data
Co-authored-by: Yuri Shkuro <yurishkuro@users.noreply.github.com>
@open-telemetry/specs-trace-approvers These are minor modifications to the text of OTEP 168 and 170, they do not substantially change the content they just spell it out more clearly and change a few terms. The reason you should review this PR is that it includes a substantial tract of text to be inserted into the specification after it merges, related to the span data model discussed in open-telemetry/opentelemetry-specification#1929. |
@open-telemetry/specs-trace-approvers please approve minor edits to the already-merged sampling OTEPs 168 and 170. |
Co-authored-by: Yuri Shkuro <yurishkuro@users.noreply.github.com>
@open-telemetry/specs-trace-approvers I would like this to merge, has several approvals already. |
Co-authored-by: Yuri Shkuro <yurishkuro@users.noreply.github.com>
…acd/170composition
OTEP #168 spawned a very long comment thread here:
#168 (comment)
This PR is an update to the text of OTEP 170 with more detail on composing probability and non-probability samplers.
This changes the name used in OTEP 170 for "head trace sampling" to "parent trace sampling" to better fit the model where spans can make independent decisions.
This uses "head trace sampling" only to describe sampling decisions being made at the root of the trace.
This changes the field returned by the Sampler to be named
log_adjusted_count
since it applies to both the new span and the new child context, not the "head".cc: @oertl