helix-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From j...@apache.org
Subject [helix] branch helix-0.9.x updated: Downgrade the log level to INFO when isInstanceSetup() fails. (#870)
Date Wed, 06 May 2020 22:06:03 GMT
This is an automated email from the ASF dual-hosted git repository.

jxue pushed a commit to branch helix-0.9.x
in repository https://gitbox.apache.org/repos/asf/helix.git


The following commit(s) were added to refs/heads/helix-0.9.x by this push:
     new a2f0cc2  Downgrade the log level to INFO when isInstanceSetup() fails. (#870)
a2f0cc2 is described below

commit a2f0cc20a0be29f3f99555a0461d065c6946230c
Author: Jiajun Wang <1803880+jiajunwang@users.noreply.github.com>
AuthorDate: Thu Mar 5 21:38:51 2020 -0800

    Downgrade the log level to INFO when isInstanceSetup() fails. (#870)
    
    Follow the log convention in ZKUtil class and downgrade the log level to INFO when isInstanceSetup()
fails. This change avoids verbose error log that misleads the users.
---
 helix-core/src/main/java/org/apache/helix/manager/zk/ZKUtil.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/helix-core/src/main/java/org/apache/helix/manager/zk/ZKUtil.java b/helix-core/src/main/java/org/apache/helix/manager/zk/ZKUtil.java
index 59e81fa..16a2002 100644
--- a/helix-core/src/main/java/org/apache/helix/manager/zk/ZKUtil.java
+++ b/helix-core/src/main/java/org/apache/helix/manager/zk/ZKUtil.java
@@ -102,7 +102,7 @@ public final class ZKUtil {
       for (String path : requiredPaths) {
         if (!zkclient.exists(path)) {
           isValid = false;
-          logger.error("Invalid instance setup, missing znode path: {}", path);
+          logger.info("Invalid instance setup, missing znode path: {}", path);
         }
       }
 


Mime
View raw message