----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24123/#review49205 -----------------------------------------------------------
Ship it! src/tests/zookeeper_tests.cpp <https://reviews.apache.org/r/24123/#comment86084> What about: Duration timeout = Milliseconds(100); - Ben Mahler On July 31, 2014, 12:26 a.m., Jiang Yan Xu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24123/ > ----------------------------------------------------------- > > (Updated July 31, 2014, 12:26 a.m.) > > > Review request for mesos and Ben Mahler. > > > Bugs: MESOS-1655 > https://issues.apache.org/jira/browse/MESOS-1655 > > > Repository: mesos-git > > > Description > ------- > > Fixed a flaky test: ZooKeeperTest.LeaderDetectorTimeoutHandling > - The original ZK session timeout was the same as AWAIT_READY timeout so it's > possible that AWAIT_READY timed out in a race. > - Plus we were spending 10 seconds clock time on this wait so I reduced the > timeout. > > > Diffs > ----- > > src/tests/zookeeper_tests.cpp be9fa06818b96e5170c68810fe16cc472f1f8b28 > > Diff: https://reviews.apache.org/r/24123/diff/ > > > Testing > ------- > > Ran the test for 350 iterations so far and it didn't fail. > > > Thanks, > > Jiang Yan Xu > >
