-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Preview: Statsdreceiver using exponential histogram #6666
Conversation
…tor-contrib into jmacd/statsdexpo
…tor-contrib into jmacd/statsdexpo2
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
@jmacd this is still in progress, right? |
Yes, this is in progress and depends on open-telemetry/opentelemetry-go#2393 merging first. I just gave a demo of this code running for my co-workers :-) |
…tor-contrib into jmacd/statsdexpo2
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
Closed as inactive. Feel free to reopen if this PR is still being worked on. |
I've been asked about this PR recently, opening it to keep people aware it's coming. |
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
I will re-open eventually. |
@jmacd any updates on this? Would love to see this get merged! |
@hardproblems will you please review the code in this PR: open-telemetry/opentelemetry-go#3022 |
In a week or two, I will re-open this PR targetting the new data structure, which is available elsewhere in the meanwhile: |
Replaced by #12951 |
…or (#6666) Signed-off-by: Bogdan Drutu <[email protected]> Signed-off-by: Bogdan Drutu <[email protected]>
Description: This is a preview implementation of the OTLP v0.11 exponential histogram being used as an alternative for the aggregation of histogram/timing measurements in the statsd receiver.
Depends on open-telemetry/opentelemetry-go#2393
Link to tracking Issue: #5742
Testing:
(I am having trouble writing tests for this code because it depends on unmerged code in the OTel-Go repository.)
Documentation: TODO