> On March 19, 2015, 3:11 p.m., Bernd Mathiske wrote: > > src/slave/containerizer/external_containerizer.cpp, line 239 > > <https://reviews.apache.org/r/32233/diff/2/?file=899898#file899898line239> > > > > dispatch(process.get(), > > &ExternalContainerizerProcess::destroy, > > containerId);
The three parameters could not be aligned one below other properly. That is why I kept process.get() on another line. And yes I did make check. - Akanksha ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32233/#review77047 ----------------------------------------------------------- On March 19, 2015, 1:47 p.m., Akanksha Agrawal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/32233/ > ----------------------------------------------------------- > > (Updated March 19, 2015, 1:47 p.m.) > > > Review request for mesos and Ben Mahler. > > > Repository: mesos > > > Description > ------- > > Replaced raw pointer by Owned pointer > > > Diffs > ----- > > src/slave/containerizer/external_containerizer.hpp > 70491375a6cd8988a33e3f18870c9170a37c0f17 > src/slave/containerizer/external_containerizer.cpp > 42c67f548caf7bddbe131e0dfa7d74227d8c2593 > > Diff: https://reviews.apache.org/r/32233/diff/ > > > Testing > ------- > > > Thanks, > > Akanksha Agrawal > >
