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

metrics: implement histogram aggregator #927

Closed
vmarchaud opened this issue Apr 6, 2020 · 1 comment
Closed

metrics: implement histogram aggregator #927

vmarchaud opened this issue Apr 6, 2020 · 1 comment
Assignees
@vmarchaud
Labels
feature-request
Projects
Metrics API and SDK

Comments

@vmarchaud
Copy link
Member

Spec: https://github.com/jmacd/opentelemetry-specification/blob/jmacd/draft_metric_sdk_spec/specification/sdk-metric.md#L249

Go implementation: https://github.com/open-telemetry/opentelemetry-go/blob/master/sdk/metric/aggregator/histogram/histogram.go
@vmarchaud vmarchaud added this to To do in Metrics API and SDK via automation Apr 6, 2020
@vmarchaud vmarchaud self-assigned this Apr 6, 2020
vmarchaud added a commit to vmarchaud/opentelemetry-js that referenced this issue Apr 6, 2020
@vmarchaud
feat(aggregators): implement histogram aggregator open-telemetry#927
a8f3010
vmarchaud added a commit to vmarchaud/opentelemetry-js that referenced this issue Apr 6, 2020
@vmarchaud
feat(aggregators): implement histogram aggregator open-telemetry#927
55a57e8
vmarchaud added a commit to vmarchaud/opentelemetry-js that referenced this issue Apr 6, 2020
@vmarchaud
feat(aggregators): implement histogram aggregator open-telemetry#927
3b35c60
vmarchaud added a commit to vmarchaud/opentelemetry-js that referenced this issue Apr 8, 2020
@vmarchaud
feat(aggregators): implement histogram aggregator open-telemetry#927
70e79c8
vmarchaud added a commit to vmarchaud/opentelemetry-js that referenced this issue Apr 9, 2020
@vmarchaud
feat(aggregators): implement histogram aggregator open-telemetry#927
234796c
vmarchaud added a commit to vmarchaud/opentelemetry-js that referenced this issue Apr 12, 2020
@vmarchaud
feat(aggregators): implement histogram aggregator open-telemetry#927
95097bb
vmarchaud added a commit to vmarchaud/opentelemetry-js that referenced this issue Apr 15, 2020
@vmarchaud
feat(aggregators): implement histogram aggregator open-telemetry#927
ac3bf14
vmarchaud added a commit to vmarchaud/opentelemetry-js that referenced this issue Apr 18, 2020
@vmarchaud
feat(aggregators): implement histogram aggregator open-telemetry#927
258198f
mayurkale22 pushed a commit that referenced this issue Apr 20, 2020
@vmarchaud
feat(aggregators): implement histogram aggregator #927 (#930)
60132b9 
* chore(metrics): move each aggregator in its own file

* feat(aggregators): implement histogram aggregator #927

* chore: address PR comments

* chore: fix ConsoleMetricExporter

* chore: address mayur comments

* chore: add documentation on internal structure
@vmarchaud
Copy link
Member Author

Landed in 0.7.0

Metrics API and SDK automation moved this from To do to Done Apr 25, 2020
@andrewthauer andrewthauer mentioned this issue Aug 18, 2021
Merged
5 tasks
pichlermarc pushed a commit to dynatrace-oss-contrib/opentelemetry-js that referenced this issue Dec 15, 2023
@dyladan
docs(redis): update readme supported versions (open-telemetry#927)
9c00bf2 
Co-authored-by: Daniel Dyla <dyladan@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vmarchaud vmarchaud

Labels
feature-request
Projects
No open projects
Metrics API and SDK
  
Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@vmarchaud