-
Notifications
You must be signed in to change notification settings - Fork 525
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
fix(storage): iter metrics should include init duration #3462
Conversation
Signed-off-by: Alex Chi <iskyzh@gmail.com>
Signed-off-by: Alex Chi <iskyzh@gmail.com>
Signed-off-by: Alex Chi <iskyzh@gmail.com>
Also we're using mininstant (thanks to @breezewish for recommendation!) for |
Codecov Report
@@ Coverage Diff @@
## main #3462 +/- ##
==========================================
- Coverage 74.48% 74.48% -0.01%
==========================================
Files 770 770
Lines 108126 108140 +14
==========================================
+ Hits 80542 80546 +4
- Misses 27584 27594 +10
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more |
One interesting finding is that iterator creation + fetching first block is taking much more time than actual scanning, which indicates some issues. maybe related to #3459 |
Signed-off-by: Alex Chi <iskyzh@gmail.com>
Signed-off-by: Alex Chi iskyzh@gmail.com
I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
Previously we only take scan duration into account, which is not accurate enough to reflect the real time taken by each iterator.
Checklist
./risedev check
(or alias,./risedev c
)Refer to a related PR or issue link (optional)