commit cc5a038b457ba04dba77e538a523bd9677641c4d
author RickyMa <rickyma@tencent.com> Tue Feb 06 14:21:12 2024 +0800
committer GitHub <noreply@github.com> Tue Feb 06 14:21:12 2024 +0800
tree bcb87f51bb606e34bcea3df455d907d4f2b5759d
parent 4dacb1e3dc50902823469c624a6ea0e94e80d636

[#1267][FOLLOWUP] improvement(client): INFO log level should be used in RetryUtils (#1500)

### What changes were proposed in this pull request?

INFO log level should be used in RetryUtils. Because these log messages are not actual errors.

### Why are the changes needed?

It's a followup PR for [#1411](https://github.com/apache/incubator-uniffle/pull/1411).

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Existing UTs.

common/src/main/java/org/apache/uniffle/common/util/RetryUtils.java

diff --git a/common/src/main/java/org/apache/uniffle/common/util/RetryUtils.java b/common/src/main/java/org/apache/uniffle/common/util/RetryUtils.java
index 961fb4f..4a15985 100644
--- a/common/src/main/java/org/apache/uniffle/common/util/RetryUtils.java
+++ b/common/src/main/java/org/apache/uniffle/common/util/RetryUtils.java
@@ -83,14 +83,14 @@
         retry++;
         if (isRetryFunc.apply(t) && retry < retryTimes) {
           if (LOG.isDebugEnabled()) {
-            LOG.error("Retry due to Throwable ", t);
+            LOG.debug("Retry due to: ", t);
           } else {
-            LOG.error(
-                "Retry due to Throwable {}. Use DEBUG level to see the full stack: {}",
+            LOG.info(
+                "Retry due to: {}. Use DEBUG level to see the full stack: {}",
                 t.getClass().getName(),
                 t.getMessage());
           }
-          LOG.error(
+          LOG.info(
               "Will retry {} more time(s) after waiting {} milliseconds.",
               retryTimes - retry,
               intervalMs);