Disable refreshMemoryBytesMetrics when running test cases on Mac (#654)

Description: Because ProcfsBasedProcessTree currently is supported only on Linux. When running test cases on Mac, the resourceCalculator is null and should be ignored. Signed-off-by: zhangjunfan <[email protected]>
tony-framework · Mar 10, 2022 · 2f5089f · 2f5089f
1 parent 1961403
commit 2f5089f
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/tony-core/src/main/java/com/linkedin/tony/TaskMonitor.java b/tony-core/src/main/java/com/linkedin/tony/TaskMonitor.java
@@ -107,6 +107,14 @@ private void refreshMetrics() {
   }
 
   private void refreshMemoryBytesMetrics() {
+    /**
+     * ProcfsBasedProcessTree currently is supported only on Linux.
+     * So when running test cases on Mac, the {@code resourceCalculator}
+     * is null and should be ignored.
+     */
+    if (resourceCalculator == null) {
+      return;
+    }
     resourceCalculator.updateProcessTree();
     double memoryBytes = resourceCalculator.getRssMemorySize();
     setMaxMetrics(MAX_MEMORY_BYTES_INDEX, memoryBytes);