----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19901/#review39523 -----------------------------------------------------------
Patch looks great! Reviews applied: [19795, 18403, 19901] All tests passed. - Mesos ReviewBot On April 4, 2014, 5:52 a.m., Till Toenshoff wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/19901/ > ----------------------------------------------------------- > > (Updated April 4, 2014, 5:52 a.m.) > > > Review request for mesos, Benjamin Hindman, Ian Downes, Niklas Nielsen, and > Vinod Kone. > > > Repository: mesos-git > > > Description > ------- > > Replaced slave::Containerizer::Termination with a protobuf message called > Termination for allowing it to be reused in ExternalContainerizer, gaining a > serializable Termination message. > > > Diffs > ----- > > include/mesos/mesos.proto 37f8a7f > src/slave/containerizer/containerizer.hpp d9ae326 > src/slave/containerizer/mesos_containerizer.hpp ee1fd30 > src/slave/containerizer/mesos_containerizer.cpp c819c97 > src/slave/slave.hpp 15e23ce > src/slave/slave.cpp a356f5f > src/tests/containerizer.hpp a9f1531 > src/tests/containerizer.cpp bfb9341 > > Diff: https://reviews.apache.org/r/19901/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Till Toenshoff > >
