----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26509/#review56357 -----------------------------------------------------------
Ship it! src/slave/main.cpp <https://reviews.apache.org/r/26509/#comment96657> When we can use EXIT(1) << please prefer it (see a few lines up for an example). Looks like you can clean up the same from your recent patch to src/master/main.cpp too! - Benjamin Hindman On Oct. 11, 2014, 1:25 a.m., Kapil Arya wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26509/ > ----------------------------------------------------------- > > (Updated Oct. 11, 2014, 1:25 a.m.) > > > Review request for mesos, Niklas Nielsen and Till Toenshoff. > > > Repository: mesos-git > > > Description > ------- > > Added --module flag for Mesos slave. > > > Diffs > ----- > > src/slave/flags.hpp 16f0cc2ab5ba16a39499608174278b3082e0585d > src/slave/main.cpp 1eafb356b58613d4ab8bd6dd245583e0d6f25a97 > > Diff: https://reviews.apache.org/r/26509/diff/ > > > Testing > ------- > > > Thanks, > > Kapil Arya > >