Optimize the log printing when multiple error occur

apache · Jun 13, 2024 · ad039e5 · ad039e5
1 parent fb8f2fb
commit ad039e5
Showing 1 changed file with 11 additions and 9 deletions.
diff --git a/iotdb-client/service-rpc/src/main/java/org/apache/iotdb/rpc/RpcUtils.java b/iotdb-client/service-rpc/src/main/java/org/apache/iotdb/rpc/RpcUtils.java
@@ -146,18 +146,20 @@ public static TSStatus getStatus(TSStatusCode tsStatusCode) {
   public static TSStatus getStatus(List<TSStatus> statusList) {
     TSStatus status = new TSStatus(TSStatusCode.MULTIPLE_ERROR.getStatusCode());
     status.setSubStatus(statusList);
-    StringBuilder errMsg = new StringBuilder().append("Multiple error occur, messages: ");
-    Set<String> msgSet = new HashSet<>();
-    for (TSStatus subStatus : statusList) {
-      if (subStatus.getCode() != TSStatusCode.SUCCESS_STATUS.getStatusCode()
-          && subStatus.getCode() != TSStatusCode.REDIRECTION_RECOMMEND.getStatusCode()) {
-        if (!msgSet.contains(status.getMessage())) {
-          errMsg.append(status.getMessage()).append("; ");
-          msgSet.add(status.getMessage());
+    if (LOGGER.isDebugEnabled()) {
+      StringBuilder errMsg = new StringBuilder().append("Multiple error occur, messages: ");
+      Set<TSStatus> msgSet = new HashSet<>();
+      for (TSStatus subStatus : statusList) {
+        if (subStatus.getCode() != TSStatusCode.SUCCESS_STATUS.getStatusCode()
+            && subStatus.getCode() != TSStatusCode.REDIRECTION_RECOMMEND.getStatusCode()) {
+          if (!msgSet.contains(status)) {
+            errMsg.append(status).append("; ");
+            msgSet.add(status);
+          }
         }
       }
+      LOGGER.debug(errMsg.toString());
     }
-    LOGGER.warn(errMsg.toString());
     return status;
   }