----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33647/#review81903 -----------------------------------------------------------
Ship it! src/slave/slave.cpp <https://reviews.apache.org/r/33647/#comment132436> Can you also comment on what the consequence of having those processes left in slave's cgroup? Maybe add a TODO here to address that? - Jie Yu On April 28, 2015, 11:33 p.m., Ian Downes wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33647/ > ----------------------------------------------------------- > > (Updated April 28, 2015, 11:33 p.m.) > > > Review request for mesos, Ben Mahler, Jie Yu, and Timothy Chen. > > > Bugs: MESOS-2668 > https://issues.apache.org/jira/browse/MESOS-2668 > > > Repository: mesos > > > Description > ------- > > Changes previous behavior where the slave would exit if processes were in the > slave cgroup. > > > Diffs > ----- > > src/slave/slave.cpp c5b4847253990b31fb05dfcd40de0102d18c8024 > > Diff: https://reviews.apache.org/r/33647/diff/ > > > Testing > ------- > > None. > > > Thanks, > > Ian Downes > >
