Re: Review Request 50642: Implemented `CgroupsIsolatorProcess::initialize`.

2016-08-02 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50642/ --- (Updated Aug. 2, 2016, 4:46 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 50642: Implemented `CgroupsIsolatorProcess::initialize`.

2016-08-02 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50642/#review144508 --- Can you rebase? - Jie Yu On July 31, 2016, 5:48 p.m., haosdent

Re: Review Request 50642: Implemented `CgroupsIsolatorProcess::initialize`.

2016-08-01 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50642/#review144457 --- Ship it! Ship It! - Jie Yu On July 31, 2016, 5:48 p.m., haos

Re: Review Request 50642: Implemented `CgroupsIsolatorProcess::initialize`.

2016-08-01 Thread Qian Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50642/#review18 --- Ship it! Ship It! - Qian Zhang On Aug. 1, 2016, 1:48 a.m., h

Re: Review Request 50642: Implemented `CgroupsIsolatorProcess::initialize`.

2016-08-01 Thread haosdent huang
> On Aug. 1, 2016, 3:10 a.m., Qian Zhang wrote: > > src/slave/containerizer/mesos/isolators/cgroups/cgroups.cpp, lines 127-132 > > > > > > Do we have to spawn each subsystem in this `initialize()` method? Or > > can

Re: Review Request 50642: Implemented `CgroupsIsolatorProcess::initialize`.

2016-07-31 Thread Qian Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50642/#review144298 --- src/slave/containerizer/mesos/isolators/cgroups/cgroups.cpp (line

Review Request 50642: Implemented `CgroupsIsolatorProcess::initialize`.

2016-07-31 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50642/ --- Review request for mesos, Gilbert Song, Jie Yu, Qian Zhang, and Zhengju Sha. Bu