Make sure uptime positive

Alluxio · Feb 21, 2020 · 30fd417 · 30fd417
1 parent 7a262f1
commit 30fd417
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/core/server/master/src/main/java/alluxio/master/metrics/DefaultMetricsMaster.java b/core/server/master/src/main/java/alluxio/master/metrics/DefaultMetricsMaster.java
@@ -141,7 +141,9 @@ protected void registerThroughputGauge(String counterName, String throughputName
         new Gauge<Object>() {
           @Override
           public Object getValue() {
-            long uptime = (mClock.millis() - mMetricsStore.getLastClearTime())
+            // Divide into two lines so uptime is always zero or positive
+            long lastClearTime = mMetricsStore.getLastClearTime();
+            long uptime = (mClock.millis() - lastClearTime)
                 / Constants.MINUTE_MS;
             long value = MetricsSystem.counter(counterName).getCount();
             // The value is bytes per minute