Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Aggregation cardinality default configuration should be experimental #3618

Closed
dashpole opened this issue Jul 24, 2023 · 0 comments · Fixed by #3619
Closed

Aggregation cardinality default configuration should be experimental #3618

dashpole opened this issue Jul 24, 2023 · 0 comments · Fixed by #3619
Assignees
@carlosalberto
Labels
spec:metrics Related to the specification/metrics directory

Comments

@dashpole
Copy link
Contributor

What are you trying to achieve?

I am trying to validate the Golang implementation of MetricReader: open-telemetry/opentelemetry-go#3662

I discovered that the default aggregation cardinality configuration on MetricReader is not marked experimental, even though all other aspects of aggregation cardinality are. I'm assuming this is an oversight. Please correct me if I am mistaken.

Additional context.

open-telemetry/opentelemetry-go#3662 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@carlosalberto carlosalberto

Labels
spec:metrics Related to the specification/metrics directory
Projects
None yet
Milestone
No milestone
2 participants
@carlosalberto @dashpole