commit 89f163fe88fc369fb08aa6c087dcb24bb943aba6
author Ian Maxon <ian@maxons.email> Fri Nov 17 00:55:40 2023 -0800
committer Ian Maxon <imaxon@uci.edu> Fri Nov 17 18:37:22 2023 +0000
tree af11c3cd36a613c21718a2ba5c41aa4873129084
parent 487dbc8d7060ce8be537cc2c52e85abe58918136

[ASTERIXDB-3278][API] Fix hit ratio calculation

- user model changes: no
- storage format changes: no
- interface changes: no

Details:
The ratio of course needs to use floating point division, not integer.

Change-Id: I568a2df4594bdde19932ba72362c9c61a96c9183
Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17948
Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Contrib: Ian Maxon <imaxon@uci.edu>
Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Reviewed-by: Wail Alkowaileet <wael.y.k@gmail.com>

asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/JobResultCallback.java

diff --git a/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/JobResultCallback.java b/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/JobResultCallback.java
index d1a1008..7ac1431 100644
--- a/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/JobResultCallback.java
+++ b/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/JobResultCallback.java
@@ -100,7 +100,7 @@
             metadata.setQueueWaitTimeInNanos(run.getJobProfile().getQueueWaitTimeInNanos());
         }
         metadata.setProcessedObjects(processedObjects);
-        metadata.setBufferCacheHitRatio(pagesRead > 0 ? (pagesRead - nonPagedReads) / pagesRead : Double.NaN);
+        metadata.setBufferCacheHitRatio(pagesRead > 0 ? (pagesRead - nonPagedReads) / (double) pagesRead : Double.NaN);
         metadata.setWarnings(AggregateWarnings);
         metadata.setTotalWarningsCount(aggregateTotalWarningsCount);
         if (run != null && run.getFlags() != null && run.getFlags().contains(JobFlag.PROFILE_RUNTIME)) {