Re: Review Request 42121: Replaced `Clock::settle()` with `os::sleep` in test case.

2016-01-12 Thread Neil Conway
> On Jan. 11, 2016, 8:09 p.m., Joseph Wu wrote: > > 3rdparty/libprocess/src/tests/metrics_tests.cpp, lines 280-283 > > > > > > Just some thoughts: > > > > The downstream event you're waiting for is a call

Re: Review Request 42121: Replaced `Clock::settle()` with `os::sleep` in test case.

2016-01-11 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42121/#review113733 --- Patch looks great! Reviews applied: [42121] Passed command:

Re: Review Request 42121: Replaced `Clock::settle()` with `os::sleep` in test case.

2016-01-11 Thread Joseph Wu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42121/#review113827 --- 3rdparty/libprocess/src/tests/metrics_tests.cpp (lines 280 - 283)