zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From eolivelli <...@git.apache.org>
Subject [GitHub] zookeeper pull request #614: ZOOKEEPER-3136 Reduce log in ClientBase in case...
Date Sat, 01 Sep 2018 08:06:41 GMT
Github user eolivelli commented on a diff in the pull request:

    https://github.com/apache/zookeeper/pull/614#discussion_r214507696
  
    --- Diff: src/java/test/org/apache/zookeeper/test/ClientBase.java ---
    @@ -287,6 +288,9 @@ public static boolean waitForServerUp(String hp, long timeout, boolean
secure) {
                             !result.contains("READ-ONLY")) {
                         return true;
                     }
    +            } catch (ConnectException e) {
    +                // ignore as this is expected, do not log stacktrace
    +                LOG.info("server {} not up", hp);
                 } catch (IOException e) {
    --- End diff --
    
    Good idea, I will log e.toString(), this will provide class name as well


---

Mime
View raw message