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
---
