ZooKeeper
  1. ZooKeeper
  2. ZOOKEEPER-1478

Small bug in QuorumTest.testFollowersStartAfterLeader( )

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 3.4.3
    • Fix Version/s: 3.4.6, 3.5.0
    • Component/s: tests
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      The following code appears in QuorumTest.testFollowersStartAfterLeader( ):

      for (int i = 0; i < 30; i++) {
      try

      { zk.create("/test", "test".getBytes(), ZooDefs.Ids.OPEN_ACL_UNSAFE, CreateMode.PERSISTENT); break; }

      catch(KeeperException.ConnectionLossException e)

      { Thread.sleep(1000); }

      // 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.");
      }

      From the comment it looks like the intention was to try to reconnect 30 times and only then trigger the Assert, but that's not what this does.
      After we fail to connect once and Thread.sleep is executed, Assert.fail will be executed without retrying create.

      1. ZOOKEEPER-1478.patch
        1 kB
        Alexander Shraer
      2. ZOOKEEPER-1478.patch
        1 kB
        Flavio Junqueira
      3. ZOOKEEPER-1478.patch
        2 kB
        Flavio Junqueira
      4. ZOOKEEPER-1478.patch
        2 kB
        Alexander Shraer
      5. ZOOKEEPER-1478.patch
        2 kB
        Mahadev konar

        Activity

        No work has yet been logged on this issue.

          People

          • Assignee:
            Alexander Shraer
            Reporter:
            Alexander Shraer
          • Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development