----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51167/#review146249 -----------------------------------------------------------
Fix it, then Ship it! src/slave/containerizer/mesos/launcher.cpp (lines 76 - 85) <https://reviews.apache.org/r/51167/#comment212604> Ditto below. Move this below 'status' src/slave/containerizer/mesos/linux_launcher.cpp (lines 80 - 89) <https://reviews.apache.org/r/51167/#comment212603> I'd adjust the order in this file so it matches the order of in the header. i.e., move this down below 'status' - Jie Yu On Aug. 20, 2016, 6:16 p.m., Kevin Klues wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51167/ > ----------------------------------------------------------- > > (Updated Aug. 20, 2016, 6:16 p.m.) > > > Review request for mesos, Benjamin Hindman, Jie Yu, and Vinod Kone. > > > Bugs: MESOS-6051 > https://issues.apache.org/jira/browse/MESOS-6051 > > > Repository: mesos > > > Description > ------- > > The functions added in this commit are not currently used anywhere in > the code base. However, they are required for some changes we are > making in order to persist container state in cases where an agent > crashes. The changes to use these function will come in subsequent > commits. > > > Diffs > ----- > > src/slave/containerizer/mesos/launcher.hpp > bf435e3a9c150648336a1becf2f075fa183428bd > src/slave/containerizer/mesos/launcher.cpp > 9efe8474a2210957ce256fc08cb35694194213c3 > src/slave/containerizer/mesos/linux_launcher.hpp > c1852226c74bc611d045be721e284141e59adcd9 > src/slave/containerizer/mesos/linux_launcher.cpp > 95dee95c5e6e613e526c92d8729ae5583c8b58f1 > src/tests/containerizer/launcher.hpp > 7e5c243efad11d04e70b36876b2ed4db82666d31 > > Diff: https://reviews.apache.org/r/51167/diff/ > > > Testing > ------- > > GTEST_FILTER="" make -j check > src/mesos-tests > sudo src/mesos-tests > > > Thanks, > > Kevin Klues > >
