-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
stats: merge non-overlapped feedback when update bucket count #10476
Conversation
Codecov Report
@@ Coverage Diff @@
## master #10476 +/- ##
==============================================
- Coverage 77.3311% 77.32% -0.0112%
==============================================
Files 413 413
Lines 87274 87262 -12
==============================================
- Hits 67490 67471 -19
+ Misses 14602 14601 -1
- Partials 5182 5190 +8 |
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.
lgtm, can you also change the test mentioned in the issue?
@winoros Added. |
/rebuild |
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.
LGTM
/run-all-tests |
What problem does this PR solve?
Fix #10365
What is changed and how it works?
When updating the bucket count, we first try to merge the max fraction of non-overlapped feedback that is contained in the bucket, the result bucket count will be more accurate than only use one feedback.
Check List
Tests
Code changes
Side effects
Related changes