-
Notifications
You must be signed in to change notification settings - Fork 647
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 ExponentialHistogram data point to metrics SDK #2421
Comments
You probably already know this Alex, but ExponentialHistogram aggregation is not in the Metrics SDK spec yet (open-telemetry/opentelemetry-specification#2252). I do think it's worth adding it to our export types now to get the OTLP exporter ready in anticipation 🙂 |
These are the major changes I'm thinking we'd need to make?:
I'm sure I'm missing small things here and there, but wanted to see if I'm on the right track before I go any further down a rabbit hole :) |
@clesleycode The majority of work on this is done by @ocelotl. The last remaining part is implemented here #2964. |
@srikanthccv should we close this then? |
No, there is one more task pending. We should close this when that gets merged. |
#2964 is merged, closing. |
The spec says there are 4 data points:
Currently the SDK only implements 3 of them: Sum, Gauge, Histogram. See:
https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/metrics/datamodel.md#opentelemetry-protocol-data-model
The text was updated successfully, but these errors were encountered: