Fix metrics when cache miss

Alluxio / alluxio

Alluxio, data orchestration for analytics and machine learning in the cloud

https://www.alluxio.io

Apache License 2.0

6.87k stars 2.94k forks source link

Open jja725 opened 1 month ago

jja725 commented 1 month ago

Fix metrics when there's cache miss

The correct behavior should be

When read from external storage (or a cache miss?), CLIENT_CACHE_PAGE_READ_CACHE_TIME_NS == 0 & CLIENT_CACHE_PAGE_READ_EXTERNAL_TIME_NS > 0
When read from cache (or cache hit?), CLIENT_CACHE_PAGE_READ_CACHE_TIME_NS > 0 & CLIENT_CACHE_PAGE_READ_EXTERNAL_TIME_NS == 0

alluxio-bot commented 1 month ago

Automated checks report:

PR title follows the conventions: FAIL
- The title of the PR does not pass all the checks. Please fix the following issues:
  - Title must not end with whitespace
Commits associated with Github account: PASS

Some checks failed. Please fix the reported issues and reply alluxio-bot, check this please to re-run checks.

alluxio-bot commented 1 month ago

Automated checks report:

All checks passed!