Github user revans2 commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/453#discussion_r168795646
--- Diff:
src/java/test/org/apache/zookeeper/server/quorum/QuorumPeerMainTest.java ---
@@ -435,7 +435,7 @@ private void waitForOne(ZooKeeper zk, States state)
throws InterruptedException
int iterations = ClientBase.CONNECTION_TIMEOUT / 500;
while (zk.getState() != state) {
if (iterations-- == 0) {
- throw new RuntimeException("Waiting too long");
+ throw new RuntimeException("Waiting too long " +
zk.getState() + " != " + state);
--- End diff --
@anmolnar and @afine I put this in for my own debugging and I forgot to
remove it. If you want me to I am happy to either remove it or file a separate
JIRA and put it up as a separate pull request, or just leave it. Either way is
fine with me.
---
