Re: Review Request 32653: Replace busy loop on ready file with a more relaxed loop

2015-04-02 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32653/#review78666 --- Ship it! Ship It! - Jie Yu On April 1, 2015, 11 p.m., Paul

Re: Review Request 32653: Replace busy loop on ready file with a more relaxed loop

2015-04-01 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32653/#review78637 --- Patch looks great! Reviews applied: [32653] All tests passed. -

Re: Review Request 32653: Replace busy loop on ready file with a more relaxed loop

2015-04-01 Thread Paul Brett
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32653/ --- (Updated April 1, 2015, 11 p.m.) Review request for mesos, Chi Zhang, Ian

Re: Review Request 32653: Replace busy loop on ready file with a more relaxed loop

2015-04-01 Thread Jie Yu
On April 1, 2015, 10:22 p.m., Jie Yu wrote: src/tests/port_mapping_tests.cpp, lines 405-406 https://reviews.apache.org/r/32653/diff/3/?file=911841#file911841line405 Please include stout/stopwatch.hpp Has this been resolved? - Jie