----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29597/#review67027 -----------------------------------------------------------
Ship it! Ship It! - Niklas Nielsen On Jan. 6, 2015, 2:39 p.m., Kapil Arya wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29597/ > ----------------------------------------------------------- > > (Updated Jan. 6, 2015, 2:39 p.m.) > > > Review request for mesos, Niklas Nielsen, Till Toenshoff, and Timothy Chen. > > > Repository: mesos-git > > > Description > ------- > > Slave flags are not being used in isolator.hpp. Removed #include > slave/flags.hpp > from isolator.hpp and included it directly in only those files that need it. > > > Diffs > ----- > > src/slave/containerizer/isolator.hpp > e82a14201f734292a46b8816c894fa8c766dae5a > src/slave/containerizer/isolators/cgroups/cpushare.hpp > 4ded0c4678ad61742a69e14233006a448f3fc3ca > src/slave/containerizer/isolators/cgroups/mem.hpp > 2fa755571b4d21b8b13301fcfd57ae05ea66e6e6 > src/slave/containerizer/isolators/cgroups/perf_event.hpp > e511c3efe4cc4ec13cc74bdcda993477a4db2e36 > src/slave/containerizer/isolators/filesystem/shared.hpp > 727f63e16fa69293c472a4b6b95fb0be04e689c8 > src/slave/containerizer/isolators/namespaces/pid.hpp > 8da6cccff04edf11fa37f5a33648821299cf49d5 > src/slave/containerizer/isolators/posix.hpp > 7a667e3e30712b98484947520b58965d3b768659 > > Diff: https://reviews.apache.org/r/29597/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Kapil Arya > >