Re: Review Request 24123: 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.

[Jiang Yan Xu]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/24123/
-----------------------------------------------------------

(Updated July 30, 2014, 5:25 p.m.)


Review request for mesos and Ben Mahler.


Changes
-------

Comments.


Summary (updated)
-----------------

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.


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 (updated)
-----

  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