> On July 24, 2012, 6:49 p.m., Benjamin Hindman wrote: > > src/linux/cgroups.cpp, line 1392 > > <https://reviews.apache.org/r/5840/diff/2/?file=123397#file123397line1392> > > > > Kill newline.
Done. > On July 24, 2012, 6:49 p.m., Benjamin Hindman wrote: > > src/linux/cgroups.cpp, line 1460 > > <https://reviews.apache.org/r/5840/diff/2/?file=123397#file123397line1460> > > > > LOG(FATAL) Done. > On July 24, 2012, 6:49 p.m., Benjamin Hindman wrote: > > src/tests/cgroups_tests.cpp, line 566 > > <https://reviews.apache.org/r/5840/diff/2/?file=123398#file123398line566> > > > > Did you want this to be a fatal thing? In the tests, use FAIL() << > > "message " << variable << " here"; Done. > On July 24, 2012, 6:49 p.m., Benjamin Hindman wrote: > > src/tests/cgroups_tests.cpp, line 575 > > <https://reviews.apache.org/r/5840/diff/2/?file=123398#file123398line575> > > > > Again, in the tests use FAIL(). Done. > On July 24, 2012, 6:49 p.m., Benjamin Hindman wrote: > > src/linux/cgroups.cpp, line 1443 > > <https://reviews.apache.org/r/5840/diff/2/?file=123397#file123397line1443> > > > > I think there is a 'discard' that takes a collection of futures (maybe > > a set not a list, but feel free to add the list version, or anything that > > can be const_iterator'ed for that matter. Done. Add a list version. - Jie ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5840/#review9403 ----------------------------------------------------------- On July 16, 2012, 7:19 p.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5840/ > ----------------------------------------------------------- > > (Updated July 16, 2012, 7:19 p.m.) > > > Review request for mesos and Benjamin Hindman. > > > Description > ------- > > The patch leverages the atomic task kill patch > https://reviews.apache.org/r/5402/ > > It can destroy an active cgroup (and all its sub-cgroups) by killing all > tasks and remove the cgroups in appropriate order. > > It uses the process::collect mechanism to parallelize the killing process. > > > Diffs > ----- > > src/linux/cgroups.hpp PRE-CREATION > src/linux/cgroups.cpp PRE-CREATION > src/tests/cgroups_tests.cpp PRE-CREATION > > Diff: https://reviews.apache.org/r/5840/diff/ > > > Testing > ------- > > On linux machines, make check. > > > Thanks, > > Jie Yu > >
