> On Jan. 20, 2015, 2:29 p.m., Kapil Arya wrote: > > src/slave/slave.hpp, line 54 > > <https://reviews.apache.org/r/30092/diff/1/?file=827312#file827312line54> > > > > Move it into a new block after the rest of "slave/XXX.hpp" block.
This is consistent with the containerizer include. Let's unify the style first. - Niklas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30092/#review68793 ----------------------------------------------------------- On Jan. 21, 2015, 2:37 p.m., Niklas Nielsen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30092/ > ----------------------------------------------------------- > > (Updated Jan. 21, 2015, 2:37 p.m.) > > > Review request for mesos and Ben Mahler. > > > Bugs: MESOS-2219 > https://issues.apache.org/jira/browse/MESOS-2219 > > > Repository: mesos-git > > > Description > ------- > > Adds resource_monitor to mesos-slave and change the slave process to use a > provided resource monitor implementation. > > > Diffs > ----- > > src/slave/flags.hpp 33a0cb639cd85ce4e03bd4ceaa3fed0834c239b9 > src/slave/main.cpp 42e46c58ab4b16033ac6a73d127968d0803490aa > src/slave/slave.hpp 70bd8c1fde4ea09fa54c76aa93424a1adb0309f6 > src/slave/slave.cpp fca83b3977b95ddda30f9830da10e124b5c605e6 > > Diff: https://reviews.apache.org/r/30092/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Niklas Nielsen > >
