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

Add support for Exponential Histograms #3324

Closed
5 tasks done
pichlermarc opened this issue Oct 12, 2022 · 3 comments · Fixed by #3504 or #3506
Closed
5 tasks done

Add support for Exponential Histograms #3324

pichlermarc opened this issue Oct 12, 2022 · 3 comments · Fixed by #3504 or #3506
Assignees
Labels
feature-request sdk:metrics Issues and PRs related to the Metrics SDK

Comments

@pichlermarc
Copy link
Member

pichlermarc commented Oct 12, 2022

Implement the Exponential Histogram Aggregation based on the specificiation.

This is an (most likely incomplete) list of things that need to be implemented for this feature:

  • implement an Aggregator
  • implement an Accumulation
  • create a representation that will be sent to the exporter via toMetricPoint() on the Aggregator that will be eventually passed to the metric exporters
  • add an static ExponentialBucketHistogramAggregation() method to the Aggregation class and implement the Aggregation modeled after the HistogramAggregation
  • add an Interface for the Exponential Histogram to the @opentelemetry/otlp-transformer package's types, so that it can be transformed to the OTLP representation (and subsequently serialized to proto or json and exported)
@pichlermarc pichlermarc added feature-request up-for-grabs Good for taking. Extra help will be provided by maintainers sdk:metrics Issues and PRs related to the Metrics SDK labels Oct 12, 2022
@pichlermarc pichlermarc added this to the Metrics After GA milestone Oct 12, 2022
@dyladan
Copy link
Member

dyladan commented Oct 19, 2022

I started on a prototype for this but it is not production ready by any means. If someone wants to work on this you can reach out to me and I can share what I have, otherwise I'll work on this after the metrics release.

@mwear
Copy link
Member

mwear commented Oct 28, 2022

I'll take this if it's still up for grabs.

@github-actions
Copy link

github-actions bot commented Jan 2, 2023

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 14 days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment