mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ben Mahler" <benjamin.mah...@gmail.com>
Subject Re: Review Request 13903: Added a cgroups::memory namespace.
Date Sat, 31 Aug 2013 00:29:54 GMT

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

(Updated Aug. 31, 2013, 12:29 a.m.)


Review request for mesos, Benjamin Hindman and Vinod Kone.


Changes
-------

Small naming change.


Repository: mesos-git


Description
-------

I've adjusted the CgroupsIsolator code to use these new functions.

Note that this always sets the soft_limit_in_bytes. This should have been done in the first
place, so let me know if you see a reason to conditionally set the soft limit as before.


Diffs (updated)
-----

  src/linux/cgroups.hpp 3989712b5b76bc960f03fdf020a4e5b735ba5d80 
  src/linux/cgroups.cpp b97a89cafab12aa642645508ad465f714ee08f1b 
  src/slave/cgroups_isolator.cpp 676768e6b8bd13820467309814845257a9c47e02 

Diff: https://reviews.apache.org/r/13903/diff/


Testing
-------

make check


Thanks,

Ben Mahler


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