Skip to content

Mark the default aggregation cardinality Experimental in MetricReader#3619

Merged
reyang merged 1 commit intoopen-telemetry:mainfrom
dashpole:experimental_agg_card
Jul 24, 2023
Merged

Mark the default aggregation cardinality Experimental in MetricReader#3619
reyang merged 1 commit intoopen-telemetry:mainfrom
dashpole:experimental_agg_card

Conversation

@dashpole
Copy link
Copy Markdown
Contributor

Fixes #3618

Changes

Mark the default aggregation cardinality limit experimental, since the rest of the aggregation cardinality limit feature is experimental.

@dashpole
Copy link
Copy Markdown
Contributor Author

cc @jmacd @jack-berg

@dashpole dashpole changed the title mark the default aggregation cardinality as experimental in MetricReader Mark the default aggregation cardinality Experimental in MetricReader Jul 24, 2023
@dashpole dashpole marked this pull request as ready for review July 24, 2023 20:45
@dashpole dashpole requested review from a team July 24, 2023 20:45
@reyang reyang added the editorial Editorial changes only (typos, changelog, ...). No content-related changes of any kind. label Jul 24, 2023
@reyang reyang merged commit fc8289b into open-telemetry:main Jul 24, 2023
@dashpole dashpole deleted the experimental_agg_card branch July 24, 2023 20:47
@jack-berg
Copy link
Copy Markdown
Member

Good catch. Thanks.

carlosalberto pushed a commit to carlosalberto/opentelemetry-specification that referenced this pull request Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

editorial Editorial changes only (typos, changelog, ...). No content-related changes of any kind.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Aggregation cardinality default configuration should be experimental

5 participants