fix cache miss metrics

Alluxio · Sep 26, 2024 · 0dbcb6f · 0dbcb6f
1 parent 6d99656
commit 0dbcb6f
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/dora/core/client/fs/src/main/java/alluxio/client/file/cache/LocalCacheManager.java b/dora/core/client/fs/src/main/java/alluxio/client/file/cache/LocalCacheManager.java
@@ -640,14 +640,11 @@ public int get(PageId pageId, int pageOffset, int bytesToRead, ReadTargetBuffer
       }
       MultiDimensionalMetricsSystem.CACHED_DATA_READ.inc(bytesRead);
       MetricsSystem.meter(MetricKey.CLIENT_CACHE_BYTES_READ_CACHE.getName()).mark(bytesRead);
-      cacheContext.incrementCounter(MetricKey.CLIENT_CACHE_BYTES_READ_CACHE.getMetricName(), BYTE,
-          bytesRead);
+      cacheContext.incrementCounter(MetricKey.CLIENT_CACHE_BYTES_READ_CACHE.getMetricName(), BYTE, bytesRead);
+      cacheContext.incrementCounter(
+          MetricKey.CLIENT_CACHE_PAGE_READ_CACHE_TIME_NS.getMetricName(), NANO, System.nanoTime() - startTime);
       LOG.debug("get({},pageOffset={}) exits", pageId, pageOffset);
       return bytesRead;
-    } finally {
-      cacheContext.incrementCounter(
-          MetricKey.CLIENT_CACHE_PAGE_READ_CACHE_TIME_NS.getMetricName(), NANO,
-          System.nanoTime() - startTime);
     }
   }