Repository: mesos Updated Branches: refs/heads/master 711797086 -> 69a8854d1
Updated a few comments in MesosContainerizer. Review: https://reviews.apache.org/r/51418 Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/69a8854d Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/69a8854d Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/69a8854d Branch: refs/heads/master Commit: 69a8854d1530897ee623ef6319d226ff60ca9198 Parents: f4894ec Author: Jie Yu <yujie....@gmail.com> Authored: Wed Aug 24 22:13:12 2016 -0700 Committer: Jie Yu <yujie....@gmail.com> Committed: Thu Aug 25 11:45:23 2016 -0700 ---------------------------------------------------------------------- src/slave/containerizer/mesos/containerizer.hpp | 21 ++++++++++---------- 1 file changed, 11 insertions(+), 10 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/mesos/blob/69a8854d/src/slave/containerizer/mesos/containerizer.hpp ---------------------------------------------------------------------- diff --git a/src/slave/containerizer/mesos/containerizer.hpp b/src/slave/containerizer/mesos/containerizer.hpp index 5da561c..078ef4f 100644 --- a/src/slave/containerizer/mesos/containerizer.hpp +++ b/src/slave/containerizer/mesos/containerizer.hpp @@ -294,31 +294,32 @@ private: // isolators and provisioner after that. Option<process::Future<Option<int>>> status; - // We keep track of the future that is waiting for the provisioner's - // `ProvisionInfo`, so that destroy will only start calling - // provisioner->destroy after provisioner->provision has finished. + // We keep track of the future for 'provisioner->provision' so + // that destroy will only start calling 'provisioner->destroy' + // after 'provisioner->provision' has finished. process::Future<ProvisionInfo> provisioning; // We keep track of the future that is waiting for all the - // isolators' prepare futures, so that destroy will only start + // 'isolator->prepare' to finish so that destroy will only start // calling cleanup after all isolators have finished preparing. process::Future<std::list<Option<mesos::slave::ContainerLaunchInfo>>> launchInfos; // We keep track of the future that is waiting for all the - // isolators' isolate futures, so that destroy will only start + // 'isolator->isolate' futures so that destroy will only start // calling cleanup after all isolators have finished isolating. process::Future<std::list<Nothing>> isolation; - // We keep track of any limitations received from each isolator so we can - // determine the cause of an executor termination. + // We keep track of any limitations received from each isolator so + // we can determine the cause of a container termination. std::vector<mesos::slave::ContainerLimitation> limitations; - // We keep track of the resources for each container so we can set the - // ResourceStatistics limits in usage(). + // We keep track of the resources for each container so we can set + // the ResourceStatistics limits in usage(). Resources resources; - // The configuration for the container to be launched. + // The configuration for the container to be launched. This field + // is only used during the launch of a container. mesos::slave::ContainerConfig config; State state;