----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12074/#review22408 -----------------------------------------------------------
Ship it! src/linux/cgroups.cpp <https://reviews.apache.org/r/12074/#comment45929> I think you'll need to wrap the ternary with parens, like monitor.cpp:189. Otherwise, this seems to print the bool status.isError(). See this: https://issues.apache.org/jira/browse/MESOS-462 src/slave/reaper.cpp <https://reviews.apache.org/r/12074/#comment45938> To be pedantic, !zombie does not imply running ;) - Ben Mahler On June 26, 2013, 6:41 a.m., Jiang Yan Xu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12074/ > ----------------------------------------------------------- > > (Updated June 26, 2013, 6:41 a.m.) > > > Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone. > > > Bugs: mesos-520 > https://issues.apache.org/jira/browse/mesos-520 > > > Repository: mesos > > > Description > ------- > > See summary. > > > Diffs > ----- > > src/linux/cgroups.cpp 5d63bde54b458e2067b9119383b3fe042c1874de > src/slave/process_isolator.cpp b750b26e73a69f0db3cac3e648dc0a77da1f1812 > src/slave/reaper.hpp 2d6db03760c25550317e79fea837f04e56fed617 > src/slave/reaper.cpp 8d3101df7a2ca1a0ab90a13dd828c072fae441ce > src/tests/reaper_tests.cpp d55ac24f323c66ab8098072d923ec528bf4bf11d > > Diff: https://reviews.apache.org/r/12074/diff/ > > > Testing > ------- > > ./bin/mesos-tests.sh --gtest_filter="ReaperTest.*" --gtest_repeat=1000 > --gtest_break_on_failure > > > Thanks, > > Jiang Yan Xu > >
