Re: Review Request 57574: Ensured the cgroup memory+swap limit is no less than the memory limit.

2017-03-17 Thread Jie Yu
> On March 14, 2017, 4:56 a.m., Kevin Klues wrote: > > src/slave/containerizer/mesos/isolators/cgroups/subsystems/memory.cpp > > Lines 266-270 (patched) > > > > > > It's not clear to me how we ensure this invariant w

Re: Review Request 57574: Ensured the cgroup memory+swap limit is no less than the memory limit.

2017-03-13 Thread Avinash sridharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57574/#review168863 --- src/slave/containerizer/mesos/isolators/cgroups/subsystems/memory

Re: Review Request 57574: Ensured the cgroup memory+swap limit is no less than the memory limit.

2017-03-13 Thread Kevin Klues
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57574/#review168864 --- src/slave/containerizer/mesos/isolators/cgroups/subsystems/memory

Re: Review Request 57574: Ensured the cgroup memory+swap limit is no less than the memory limit.

2017-03-13 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57574/#review168856 --- Ship it! Ship It! - haosdent huang On March 14, 2017, 12:04

Review Request 57574: Ensured the cgroup memory+swap limit is no less than the memory limit.

2017-03-13 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57574/ --- Review request for mesos, Benjamin Mahler, haosdent huang, Ian Downes, and Kevin