-
Notifications
You must be signed in to change notification settings - Fork 206
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
2.1 prefetch metrics #892
2.1 prefetch metrics #892
Conversation
Signed-off-by: Changwei Ge <gechangwei@bytedance.com>
storage/src/cache/worker.rs
Outdated
@@ -25,6 +25,8 @@ use crate::cache::{BlobCache, BlobIoRange}; | |||
use crate::factory::ASYNC_RUNTIME; | |||
use crate::RAFS_MAX_CHUNK_SIZE; | |||
|
|||
static PREFETCH_BEGIN_TIMING: Once = Once::new(); |
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.
This is a global variable, so it will only be executed once.
How about the case when prefetching multiple images?
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.
Thanks for your review, it's a good catch. Already fixed it.
In addition, added some comments telling the meaning of the newly added metrics and how to leverage them to estimate prefetch performance.
Record prefech request average latency. Calculate prefetch average bandwidth. Signed-off-by: Changwei Ge <gechangwei@bytedance.com>
Signed-off-by: Changwei Ge <gechangwei@bytedance.com>
4c48504
to
c7b3f89
Compare
/retest |
@changweige , the test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/35563 |
@changweige , The CI test is completed, please check result:
Sorry, your test job failed. Please get the details in the link. |
/retest |
@jiangliu , the test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/35685 |
/retest |
@yqleng1987 , the test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/35726 |
@yqleng1987 , The CI test is completed, please check result:
Congratulations, your test job passed! |
Please also open a PR for master |
Sure. :-) |
Record prefetch request average latency.
Calculate prefetch average bandwidth.