[IOTDB-4373] Fix statistic of disk in metric. (apache#7295)

yonyou-tsdb · Sep 14, 2022 · 8f9e28b · 8f9e28b
1 parent 06a599c
commit 8f9e28b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/server/src/main/java/org/apache/iotdb/db/service/metrics/predefined/FileMetrics.java b/server/src/main/java/org/apache/iotdb/db/service/metrics/predefined/FileMetrics.java
@@ -101,7 +101,7 @@ public void bindTo(AbstractMetricService metricService) {
         "unseq");
 
     // finally start to update the value of some metrics in async way
-    if (metricService.isEnable() && null != currentServiceFuture) {
+    if (metricService.isEnable() && null == currentServiceFuture) {
       currentServiceFuture =
           ScheduledExecutorUtil.safelyScheduleAtFixedRate(
               service,
@@ -118,7 +118,7 @@ public void bindTo(AbstractMetricService metricService) {
   public void unbindFrom(AbstractMetricService metricService) {
     // first stop to update the value of some metrics in async way
     if (currentServiceFuture != null) {
-      currentServiceFuture.cancel(false);
+      currentServiceFuture.cancel(true);
       currentServiceFuture = null;
     }
 

diff --git a/server/src/main/java/org/apache/iotdb/db/service/metrics/predefined/SystemMetrics.java b/server/src/main/java/org/apache/iotdb/db/service/metrics/predefined/SystemMetrics.java
@@ -54,7 +54,7 @@ public void bindTo(AbstractMetricService metricService) {
     collectSystemMemInfo(metricService);
 
     // finally start to update the value of some metrics in async way
-    if (metricService.isEnable() && null != currentServiceFuture) {
+    if (metricService.isEnable() && null == currentServiceFuture) {
       currentServiceFuture =
           ScheduledExecutorUtil.safelyScheduleAtFixedRate(
               service,
@@ -71,7 +71,7 @@ public void bindTo(AbstractMetricService metricService) {
   public void unbindFrom(AbstractMetricService metricService) {
     // first stop to update the value of some metrics in async way
     if (currentServiceFuture != null) {
-      currentServiceFuture.cancel(false);
+      currentServiceFuture.cancel(true);
       currentServiceFuture = null;
     }