mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Kapil Arya" <ka...@mesosphere.io>
Subject Re: Review Request 29597: Removes #include slave/flags.hpp from isolator.hpp
Date Sat, 10 Jan 2015 07:25:15 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29597/
-----------------------------------------------------------

(Updated Jan. 10, 2015, 2:25 a.m.)


Review request for mesos, Niklas Nielsen, Till Toenshoff, and Timothy Chen.


Changes
-------

rebased diff.


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 (updated)
-----

  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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message