[IOTDB-6351] Don't retry if operation has already succeed even if the…

… DN is not running status
apache · Sep 21, 2024 · a37b0ed · a37b0ed
1 parent 0ea84ca
commit a37b0ed
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/iotdb-core/node-commons/src/main/java/org/apache/iotdb/commons/utils/StatusUtils.java b/iotdb-core/node-commons/src/main/java/org/apache/iotdb/commons/utils/StatusUtils.java
@@ -206,13 +206,16 @@ public static TSStatus getStatus(TSStatusCode statusCode) {
   }
 
   public static boolean needRetry(TSStatus status) {
+    // succeed operation should never retry
+    if (status == null || status.getCode() == TSStatusCode.SUCCESS_STATUS.getStatusCode()) {
+      return false;
+    }
     // always retry while node is in not running case
     if (!COMMON_CONFIG.isRunning()) {
       return true;
-    } else if (status == null) {
-      return false;
+    } else {
+      return needRetryHelper(status);
     }
-    return needRetryHelper(status);
   }
 
   public static boolean needRetryHelper(TSStatus status) {