> On Dec. 1, 2017, 11:32 p.m., Qian Zhang wrote: > > src/slave/containerizer/mesos/containerizer.hpp > > Lines 182-184 (original), 183-185 (patched) > > <https://reviews.apache.org/r/64264/diff/1/?file=1906557#file1906557line183> > > > > Not yours. Can we swap these two lines to make it consistent with the > > declaration in `MesosContainerizer`.
I would prefer the swap them in `MesosContainerizer` class. Let me fix them in a followup patch. - Gilbert ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64264/#review192616 ----------------------------------------------------------- On Dec. 1, 2017, 5:10 p.m., Gilbert Song wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/64264/ > ----------------------------------------------------------- > > (Updated Dec. 1, 2017, 5:10 p.m.) > > > Review request for mesos, Jie Yu, Qian Zhang, Vinod Kone, and Zhitao Li. > > > Bugs: MESOS-8295 > https://issues.apache.org/jira/browse/MESOS-8295 > > > Repository: mesos > > > Description > ------- > > Added excluded image parameter to containerizer pruneImages(). > > > Diffs > ----- > > src/slave/containerizer/composing.hpp > 3d006093f484308311e054fdd1d3d27bab3f37cd > src/slave/containerizer/composing.cpp > 83880962c004e259619b123db8196ee8293683a8 > src/slave/containerizer/containerizer.hpp > 7a0c6fc6452325a318d9e1c91874098e401bc1ea > src/slave/containerizer/docker.hpp 9df98499bb7f334d8595e22ac93126453296a0f2 > src/slave/containerizer/docker.cpp d837442ca073d6872c94a9332599423c005a8874 > src/slave/containerizer/mesos/containerizer.hpp > e2739e017cb8dda37d94ad809ca1bd461f308bfb > src/slave/containerizer/mesos/containerizer.cpp > 44e8f7acc0302f68cd237bc27fe01131a1475711 > src/tests/containerizer.hpp 3d162fecf03baf4747fa8b833fb0292d55195003 > src/tests/containerizer.cpp 665bd5d3968e47c243b5a47589715d1649066183 > src/tests/containerizer/mock_containerizer.hpp > bbfa7681a874158caf085d132b16b59b74a0c4ab > > > Diff: https://reviews.apache.org/r/64264/diff/1/ > > > Testing > ------- > > make check > > > Thanks, > > Gilbert Song > >
