-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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 the Exponential Histogram Aggregator. #4245
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #4245 +/- ##
=======================================
+ Coverage 83.0% 83.2% +0.1%
=======================================
Files 217 218 +1
Lines 17166 17460 +294
=======================================
+ Hits 14264 14538 +274
- Misses 2611 2629 +18
- Partials 291 293 +2
|
Fix spelling error
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we add a "test example" to demonstrate how one can configure an exponential histogram?
Done: added in b41b1e4 |
Changelog is missing 😉 |
Co-authored-by: Robert Pająk <pellared@hotmail.com>
Expo Histogram has the same compatibility as Explicit Bucket Histograms
Moved Expo Histo to match new style
Can you add "inOut" to the ignored codespell words? |
This is the final part of #2966.
This adds the Aggregator and the mechanism to configure it.
Some notes when reading this: