----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24123/#review49186 -----------------------------------------------------------
src/tests/zookeeper_tests.cpp <https://reviews.apache.org/r/24123/#comment86048> What's stopping it from being even lower? Say, 100 milliseconds? - Ben Mahler On July 30, 2014, 10:44 p.m., Jiang Yan Xu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24123/ > ----------------------------------------------------------- > > (Updated July 30, 2014, 10:44 p.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 > >
