----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10231/#review18665 -----------------------------------------------------------
third_party/libprocess/third_party/stout/include/stout/os.hpp <https://reviews.apache.org/r/10231/#comment39043> We don't use periods at the end of log lines. third_party/libprocess/third_party/stout/include/stout/os.hpp <https://reviews.apache.org/r/10231/#comment39044> We wrap comments at 70 column width and code at 80! third_party/libprocess/third_party/stout/tests/os_tests.cpp <https://reviews.apache.org/r/10231/#comment39045> Wrap this in ASSERT_SOME. third_party/libprocess/third_party/stout/tests/os_tests.cpp <https://reviews.apache.org/r/10231/#comment39046> ditto third_party/libprocess/third_party/stout/tests/os_tests.cpp <https://reviews.apache.org/r/10231/#comment39047> ditto. - Vinod Kone On April 1, 2013, 11:47 p.m., Jiang Yan Xu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10231/ > ----------------------------------------------------------- > > (Updated April 1, 2013, 11:47 p.m.) > > > Review request for mesos, Benjamin Hindman, Vinod Kone, and Ben Mahler. > > > Description > ------- > > This is part 1. > Part 2 would be to replace sleep() call in Mesos to use os::sleep(). > > > This addresses bug MESOS-415. > https://issues.apache.org/jira/browse/MESOS-415 > > > Diffs > ----- > > third_party/libprocess/third_party/stout/include/stout/os.hpp > 5c217836a4a60402945544cba862b4c936ff0f83 > third_party/libprocess/third_party/stout/tests/os_tests.cpp > d543b6d585097fbd6ee455d4807f23eb3b532a23 > > Diff: https://reviews.apache.org/r/10231/diff/ > > > Testing > ------- > > Ran make check. > > > Thanks, > > Jiang Yan Xu > >
