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.

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

Histogram min/max support #3010

Closed
1 task
bryan-aguilar opened this issue May 31, 2022 · 1 comment · Fixed by #3032
Closed
1 task

Histogram min/max support #3010

bryan-aguilar opened this issue May 31, 2022 · 1 comment · Fixed by #3032
Assignees
Milestone

Comments

@bryan-aguilar
Copy link

  • [ x] This only affects the JavaScript OpenTelemetry library
  • This may affect other libraries, but I would like to get opinions here first

Does the metrics sdk support Histogram min/max values from the data model? I was poking around the sdk and did not see it in the Aggregator but I don't have the most experience with JS codebases.

Maybe related: #2124

@dyladan
Copy link
Member

dyladan commented May 31, 2022

Hmm I thought min/max was optional but it appears it's a required optional feature. Must have missed that. https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/metrics/sdk.md#explicit-bucket-histogram-aggregation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants