----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5402/#review9402 -----------------------------------------------------------
Update this review with the same comments as https://reviews.apache.org/r/5401 and then I'll do another pass! src/linux/cgroups.hpp <https://reviews.apache.org/r/5402/#comment20139> Let's just thread the default from the beginning, and use a well typed duration such as 'seconds'. src/linux/cgroups.cpp <https://reviews.apache.org/r/5402/#comment20140> Kill newline. - Benjamin Hindman On July 16, 2012, 7:10 p.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5402/ > ----------------------------------------------------------- > > (Updated July 16, 2012, 7:10 p.m.) > > > Review request for mesos, Benjamin Hindman and Vinod Kone. > > > Description > ------- > > This patch leverages the freezer subsystem in cgroups to kill all the > processes in a cgroup atomically. > > The main idea is to freeze all the processes in a cgroup first, then send > kill signal to all the proceses. This avoids the need of walking the proc > process tree to kill all processes associated with an executor. In fact, the > original killtree solution assumes that the user processes haven't blocked > the SIGSTOP signal, which may not be true in some cases. > > > Diffs > ----- > > src/linux/cgroups.hpp cec9e36 > src/linux/cgroups.cpp 93c140f > src/tests/cgroups_tests.cpp 7f0f901 > > Diff: https://reviews.apache.org/r/5402/diff/ > > > Testing > ------- > > On Linux machine, make check. > > > Thanks, > > Jie Yu > >
