Re: Review Request 16586: When the detectors (ZooKeeperMasterDetector, LeaderDetector) fail, subsequent detect() calls now get failed Futures directly.

[Jiang Yan Xu]

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

(Updated Jan. 14, 2014, 4:47 p.m.)


Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone.


Changes
-------

Addressed comments.


Bugs: MESOS-892
    https://issues.apache.org/jira/browse/MESOS-892


Repository: mesos-git


Description
-------

- Previously a pending future is returned to the client who calls 
Detector::detect() after the detector enters a non-operational state due to a 
non-retryable error in the last detect() call.


Diffs (updated)
-----

  src/master/detector.cpp 532e048924789d6a02bb8769a5429d67010d02cd 
  src/tests/master_contender_detector_tests.cpp 
d532c17c64c4c797053616cb486a907855bb723b 
  src/zookeeper/detector.cpp 6723d036bbd2ea814c81d777d81b2fc0b261a0fd 

Diff: https://reviews.apache.org/r/16586/diff/


Testing
-------

Added tests.
make check.


Thanks,

Jiang Yan Xu