From 05f1748c138ec2073caba0a4a06ec99d5c489aa7 Mon Sep 17 00:00:00 2001 From: Kayla Reopelle <87386821+kaylareopelle@users.noreply.github.com> Date: Wed, 1 Oct 2025 16:06:26 -0700 Subject: [PATCH] Update Probability Sampler description When reading through #4267, I noticed that the Probability Sampler description still had a reference to ratio-based. I think that entire section may be cut since the clarifications were specifically related to the old name. Also, found a small typo. --- specification/trace/sdk.md | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/specification/trace/sdk.md b/specification/trace/sdk.md index 12ccfac020b..4542b9cfcee 100644 --- a/specification/trace/sdk.md +++ b/specification/trace/sdk.md @@ -512,14 +512,7 @@ OpenTelemetry follows W3C Trace Context Level 2, which specifies 56 bits of rand The `ProbabilitySampler` sampler MUST ignore the parent `SampledFlag`. For respecting the parent `SampledFlag`, see the `ParentBased` sampler specified below. -Note that the "ratio-based" part of this Sampler's name implies that -it makes a probability decision directly from the TraceID, even though -it was not originally specified in an exact way. In the present -specification, the Sampler decision is more nuanced: only a portion of -the identifier is used, after checking whether the OpenTelemetry -TraceState field contains an explicit randomness value. - -Note that this is a non-composable form of probaiblity +Note that this is a non-composable form of probability sampler. `ProbabilitySampler` directly implements the SDKs Sampler API, whereas [`ComposableProbability`](#composableprobability) is the composable form for use with [`CompositeSampler`](#compositesampler).