ZOOKEEPER-1104. CLONE - In QuorumTest, use the same "for ( .. try { break } catch { } )" pattern in testFollowersStartAfterLeaders as in testSessionMove. (Eugene Koontz via mahadev)

git-svn-id: https://svn.apache.org/repos/asf/zookeeper/trunk@1157690 13f79535-47bb-0310-9956-ffa450edef68

CHANGES.txt

@@ -383,6 +383,10 @@ IMPROVEMENTS:
 
   ZOOKEEPER-1143. quorum send & recv workers are missing thread names
   (phunt via mahadev)
+  
+  ZOOKEEPER-1104. CLONE - In QuorumTest, use the same "for ( .. try { break }
+  catch { } )" pattern in testFollowersStartAfterLeaders as in testSessionMove. 
+  (Eugene Koontz via mahadev)
 
 NEW FEATURES:
   ZOOKEEPER-729. Java client API to recursively delete a subtree.

src/java/test/org/apache/zookeeper/test/QuorumTest.java

@@ -321,18 +321,14 @@ public class QuorumTest extends ZKTestCase {
                         "127.0.0.1:" + qu.getPeer(2).clientPort,
                         CONNECTION_TIMEOUT));
 
-        boolean success = false;
         for (int i = 0; i < 30; i++) {
             try {
                 zk.create("/test", "test".getBytes(), ZooDefs.Ids.OPEN_ACL_UNSAFE,
                     CreateMode.PERSISTENT);
-                success = true;
                 break;
             } catch(KeeperException.ConnectionLossException e) {
                 Thread.sleep(1000);
             }
-        }
-        if (!success) {
             // test fails if we still can't connect to the quorum after 30 seconds.
             Assert.fail("client could not connect to reestablished quorum: giving up after 30+ seconds.");
         }