-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17805/
-----------------------------------------------------------
(Updated Feb. 7, 2014, 2:21 p.m.)
Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone.
Changes
-------
BenM's and BenH's
Bugs: MESOS-837
https://issues.apache.org/jira/browse/MESOS-837
Repository: mesos-git
Description
-------
- Note that this fix acts as a safeguard to prevent AWAIT_* in tests running
forever. It does not eliminate the need to advance the clock in loops in some
tests.
- I found putting Clock::advance() (optionally with os::sleep) in AWAIT_* too
implicit and takes control over the clock from developers who write tests. (See
related ticket https://issues.apache.org/jira/browse/MESOS-827, will explain
elsewhere.)
Diffs (updated)
-----
3rdparty/libprocess/include/process/gtest.hpp
753ef74f422f95add9fadd96951ffdeeceddb7c5
Diff: https://reviews.apache.org/r/17805/diff/
Testing
-------
- make check
- some modified tests in https://reviews.apache.org/r/17806
- something like below (not added to review)
Clock::pause();
Future<Nothing> f;
// Fail after 10 seconds.
AWAIT_READY(f);
Clock::resume();
Thanks,
Jiang Yan Xu
