Re: Review Request 45492: Used ChildHooks in Mesos [2/5].

2016-04-07 Thread Joerg Schad
> On April 5, 2016, 6:48 p.m., Joris Van Remoortere wrote: > > src/slave/containerizer/external_containerizer.cpp, lines 1091-1092 > > > > > > why not inline this? I couldn't find a nice/readable way to inline the

Re: Review Request 45492: Used ChildHooks in Mesos [2/5].

2016-04-05 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45492/#review127177 --- Fix it, then Ship it! src/slave/containerizer/docker.cpp

Re: Review Request 45492: Used ChildHooks in Mesos [2/5].

2016-03-31 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45492/ --- (Updated March 31, 2016, 10:21 a.m.) Review request for mesos and Joris Van