Mark base-2 exponential histogram as stable by jack-berg · Pull Request #3041 · open-telemetry/opentelemetry-specification (original) (raw)

Conversation

Is this pending on #3017?

I think #3017 is nice to have, but not blocking since its a backwards compatible addition.

Circling back on Jack's last question: should the expo aggregation be a SHOULD or a MUST for each language?

cc @jsuereth @jmacd @reyang

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

I'm ok with SHOULD or MUST. For SDKs that need help implementing exponential histogram, happy to pitch in.

I've changed the aggregation name to "Base2 Exponential Bucket Histogram Aggregation" per this conversation.

Note that the metric point defined in data-model.md is still named ExponentialHistogram, since the same point could be reused with a base 10 exponential histogram aggregation.

reyang changed the titleMark exponential histogram as stable Mark base-2 exponential histogram as stable

Jan 13, 2023

carlosalberto pushed a commit to carlosalberto/opentelemetry-specification that referenced this pull request

Oct 31, 2024

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})

Reviewers

jmacd jmacd approved these changes

MrAlias MrAlias left review comments

reyang reyang approved these changes

jsuereth jsuereth approved these changes

carlosalberto carlosalberto approved these changes

+1 more reviewer

pirgeo pirgeo left review comments

Reviewers whose approvals may not affect merge requirements

Labels