Skip to content

[chore][processor/dynamic_sampling] expand documentation for alpha - #50041

Merged
songy23 merged 3 commits into
open-telemetry:mainfrom
MikeGoldsmith:mike/dynamicsampling-docs
Aug 4, 2026
Merged

[chore][processor/dynamic_sampling] expand documentation for alpha#50041
songy23 merged 3 commits into
open-telemetry:mainfrom
MikeGoldsmith:mike/dynamicsampling-docs

Conversation

@MikeGoldsmith

@MikeGoldsmith MikeGoldsmith commented Aug 4, 2026

Copy link
Copy Markdown
Member

Description

Closes out the documentation items on the alpha tracking issue:

  • The "How it works" section now states explicitly that samplers only ever produce rates and that the keep/drop decision is a single mechanism for every sampler type, the processor's rate-to-threshold conversion checked against the trace's randomness. A note contrasts this with dynsampler-go's own samplers (e.g. its DeterministicSampler hash check), which this processor never uses.
  • A new "Worked examples" section with two complete configurations: error retention with an adaptive default rule, and a throughput-bounded fleet, each with sizing guidance.

Docs only, no behavior change.

Link to tracking issue

Refs #49311

Testing

The OTTL condition used in the worked examples is covered by the existing README example compile test. Full test suite and lint pass.

Documentation

This PR is documentation.

Authorship

  • I, a human, wrote this pull request description myself.

…examples, config migration

Assisted-by: Claude Fable 5
@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Aug 4, 2026

Copy link
Copy Markdown

Pull request dashboard status

Merged · refreshed 2026-08-04 22:32 UTC

Status above doesn't look right?
  • Anything look wrong? Report it with what you expected; it helps us improve the dashboard.

@songy23
songy23 merged commit 5ee3607 into open-telemetry:main Aug 4, 2026
101 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants