helix-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jiajunw...@apache.org
Subject [helix] branch master updated: Downgrade the log level to INFO when isInstanceSetup() fails. (#870)
Date Fri, 06 Mar 2020 05:39:01 GMT
This is an automated email from the ASF dual-hosted git repository.

jiajunwang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/helix.git


The following commit(s) were added to refs/heads/master by this push:
     new 3e264f0  Downgrade the log level to INFO when isInstanceSetup() fails. (#870)
3e264f0 is described below

commit 3e264f04ebec8ffaa756595e811da1f69ab39d55
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 126ae38..1c5784f 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
@@ -146,7 +146,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