You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@helix.apache.org by ji...@apache.org on 2020/03/06 05:39:01 UTC
[helix] branch master updated: Downgrade the log level to INFO when
isInstanceSetup() fails. (#870)
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 <18...@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);
}
}