-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8096/
-----------------------------------------------------------
(Updated Feb. 9, 2013, 12:05 a.m.)
Review request for mesos, Benjamin Hindman and Ben Mahler.
Changes
-------
No need to re-review. Just uploading a rebased diff for posterity.
Submitting now.
Description
-------
Now, we do the executor setup (fetching etc) inside the forked process but
before assigning the process to its own cgroup.
This accomplishes two things:
--> unblocks the cgroups isolation module from any potential hangs during
executor setup
--> does not charge (for memory) the executor for the setup
This addresses bugs MESOS-262 and MESOS-310.
https://issues.apache.org/jira/browse/MESOS-262
https://issues.apache.org/jira/browse/MESOS-310
Diffs (updated)
-----
src/slave/cgroups_isolation_module.cpp 63cefc3
src/slave/process_based_isolation_module.cpp 3d50a4b
Diff: https://reviews.apache.org/r/8096/diff/
Testing
-------
sudo make check
Thanks,
Vinod Kone
