opentelemetry-specification icon indicating copy to clipboard operation
opentelemetry-specification copied to clipboard

4566 Clarifications for Composite Samplers proposal

Open PeterF778 opened this issue 5 months ago • 1 comments

Addressing issue #4566.

Fixes #4566

Changes

Clarifying the meaning of IsAjustedCountReliable, providing guidance for selecting the Randomness value R for sampling decision making.

  • [ ] Related issues #4566
  • [ ] Related OTEP(s) # OTEP-0250

PeterF778 avatar Jun 24 '25 20:06 PeterF778

We usually don't update OTEPs, and instead clarify or refine things in the actual Spec contents. Probably worth checking this clarification exists in the Spec (too)? Wdyt @jmacd ?

carlosalberto avatar Jun 25 '25 15:06 carlosalberto

I would feel comfortable adjusting this in the specification directly. It seems like a very small change to write a new OTEP.

jmacd avatar Jul 01 '25 15:07 jmacd

This PR was marked stale due to lack of activity. It will be closed in 7 days.

github-actions[bot] avatar Jul 09 '25 03:07 github-actions[bot]

Closed as inactive. Feel free to reopen if this PR is still being worked on.

github-actions[bot] avatar Jul 16 '25 03:07 github-actions[bot]

@kentquirk @jmacd @oertl ping ;)

carlosalberto avatar Jul 24 '25 15:07 carlosalberto

This PR was marked stale due to lack of activity. It will be closed in 7 days.

github-actions[bot] avatar Aug 08 '25 03:08 github-actions[bot]

https://github.com/open-telemetry/opentelemetry-specification/pull/4569/commits/d1890766b1241cbaa39b4da4cf52de98be190f7f

jmacd avatar Aug 14 '25 15:08 jmacd