-
Notifications
You must be signed in to change notification settings - Fork 821
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
feat: Add Exponential Histogram #3498
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #3498 +/- ##
==========================================
+ Coverage 93.74% 93.95% +0.21%
==========================================
Files 249 256 +7
Lines 7593 8057 +464
Branches 1582 1673 +91
==========================================
+ Hits 7118 7570 +452
- Misses 475 487 +12
|
Based on the discussion at the SIG meeting earlier today, I split this PR into three parts. See: I'll leave this draft PR open so folks can see the three parts combined. |
This PR 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. |
I'm closing this PR as all three parts (see below) have merged 🎉 Thank you again, @mwear, for putting in all the work and sticking with us during the reviews 🙂 |
Which problem is this PR solving?
This PR includes an implementation of the Exponential Histogram for Javascript. It is heavily based on the Golang Reference Implementation.
Fixes #3324
Short description of the changes
This PR contains the entire implementation of the Exponential Histogram, but can be considered in three parts. They are:
Note: I will give a walkthrough of this PR at the JS SIG meeting on Dec 21.
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist: