Skip to content

Emit warnings when TraceIdRatioBased Sampler is used as child sampler #7790

@jack-berg

Description

@jack-berg

Implement this portion of the spec: https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/trace/sdk.md#compatibility-warnings-for-traceidratiobased-sampler

Context:

This doesn't touch the top-level, where TraceIdRatio has a new development spec to output deprecation warnings - I think this will need to be implemented when the spec is marked stable, though not sure.

Yeah let's handle that separately. Need to think through what "in development" means in terms of warnings. Can we really recommend upgrading to unstable ProbabilitySampler?

Originally posted by @jack-berg in #7786 (comment)

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions