You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by vi...@apache.org on 2016/10/07 19:38:54 UTC

[1/3] mesos git commit: Improved mesos.proto comments for partition-aware frameworks.

Repository: mesos
Updated Branches:
  refs/heads/master bb8b14456 -> 0f97117ba


Improved mesos.proto comments for partition-aware frameworks.

Review: https://reviews.apache.org/r/52632/


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/f5b6a380
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/f5b6a380
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/f5b6a380

Branch: refs/heads/master
Commit: f5b6a3809d449772e1112d80cf057db3c020cd62
Parents: bb8b144
Author: Neil Conway <ne...@gmail.com>
Authored: Fri Oct 7 12:38:30 2016 -0700
Committer: Vinod Kone <vi...@gmail.com>
Committed: Fri Oct 7 12:38:30 2016 -0700

----------------------------------------------------------------------
 include/mesos/mesos.proto    | 42 +++++++++++++++++++--------------------
 include/mesos/v1/mesos.proto | 42 +++++++++++++++++++--------------------
 2 files changed, 42 insertions(+), 42 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/f5b6a380/include/mesos/mesos.proto
----------------------------------------------------------------------
diff --git a/include/mesos/mesos.proto b/include/mesos/mesos.proto
index 2a3c251..7d0201e 100644
--- a/include/mesos/mesos.proto
+++ b/include/mesos/mesos.proto
@@ -297,17 +297,18 @@ message FrameworkInfo {
       // Receive offers with resources that are shared.
       SHARED_RESOURCES = 4;
 
-      // Indicates that the framework is prepared to handle the
+      // Indicates that (1) the framework is prepared to handle the
       // following TaskStates: TASK_UNREACHABLE, TASK_DROPPED,
-      // TASK_GONE, TASK_GONE_BY_OPERATOR, and TASK_UNKNOWN.
+      // TASK_GONE, TASK_GONE_BY_OPERATOR, and TASK_UNKNOWN, and (2)
+      // the framework will assume responsibility for managing
+      // partitioned tasks that reregister with the master.
       //
-      // With this capability, frameworks can define how they would
-      // like to handle partitioned tasks. Frameworks will receive
-      // TASK_UNREACHABLE for tasks on partitioned agents; if/when the
-      // partitioned agent reregisters, the task will not be killed.
-      // Frameworks that enable this capability will never receive
-      // TASK_LOST; they will receive one of the most specific task
-      // statuses listed above instead.
+      // Frameworks that enable this capability can define how they
+      // would like to handle partitioned tasks. Frameworks will
+      // receive TASK_UNREACHABLE for tasks on agents that are
+      // partitioned from the master. If/when a partitioned agent
+      // reregisters, tasks on the agent that were started by
+      // PARTITION_AWARE frameworks will not killed.
       //
       // Without this capability, frameworks will receive TASK_LOST
       // for tasks on partitioned agents; such tasks will be killed by
@@ -1517,8 +1518,8 @@ enum TaskState {
   TASK_KILLED = 4;   // TERMINAL: The task was killed by the executor.
   TASK_ERROR = 7;    // TERMINAL: The task description contains an error.
 
-  // This is only sent when the framework does NOT opt-in to the
-  // PARTITION_AWARE capability.
+  // In Mesos 1.2, this will only be sent when the framework does NOT
+  // opt-in to the PARTITION_AWARE capability.
   TASK_LOST = 5;     // TERMINAL: The task failed but can be rescheduled.
 
   // The following task statuses are only sent when the framework
@@ -1527,8 +1528,8 @@ enum TaskState {
   // The task failed to launch because of a transient error. The
   // task's executor never started running. Unlike TASK_ERROR, the
   // task description is valid -- attempting to launch the task again
-  // may be successful. This is a terminal state.
-  TASK_DROPPED = 9;
+  // may be successful.
+  TASK_DROPPED = 9;  // TERMINAL.
 
   // The task was running on an agent that has lost contact with the
   // master, typically due to a network failure or partition. The task
@@ -1538,23 +1539,22 @@ enum TaskState {
   // The task was running on an agent that has been shutdown (e.g.,
   // the agent become partitioned, rebooted, and then reconnected to
   // the master; any tasks running before the reboot will transition
-  // from UNREACHABLE to GONE). The task is no longer running. This is
-  // a terminal state.
-  TASK_GONE = 11;
+  // from UNREACHABLE to GONE). The task is no longer running.
+  TASK_GONE = 11;    // TERMINAL.
 
   // The task was running on an agent that the master cannot contact;
   // the operator has asserted that the agent has been shutdown, but
   // this has not been directly confirmed by the master. If the
   // operator is correct, the task is not running and this is a
-  // terminal state; if the operator is mistaken, the task might still
-  // be running, and might return to the RUNNING state in the future.
+  // terminal state; if the operator is mistaken, the task may still
+  // be running and might return to RUNNING in the future.
   TASK_GONE_BY_OPERATOR = 12;
 
   // The master has no knowledge of the task. This is typically
   // because either (a) the master never had knowledge of the task, or
-  // (b) the master forgot about the task because it garbaged
-  // collected its metadata about the task. The task may or may not
-  // still be running.
+  // (b) the master forgot about the task because it garbage collected
+  // its metadata about the task. The task may or may not still be
+  // running.
   TASK_UNKNOWN = 13;
 }
 

http://git-wip-us.apache.org/repos/asf/mesos/blob/f5b6a380/include/mesos/v1/mesos.proto
----------------------------------------------------------------------
diff --git a/include/mesos/v1/mesos.proto b/include/mesos/v1/mesos.proto
index 57846a5..212321b 100644
--- a/include/mesos/v1/mesos.proto
+++ b/include/mesos/v1/mesos.proto
@@ -297,17 +297,18 @@ message FrameworkInfo {
       // Receive offers with resources that are shared.
       SHARED_RESOURCES = 4;
 
-      // Indicates that the framework is prepared to handle the
+      // Indicates that (1) the framework is prepared to handle the
       // following TaskStates: TASK_UNREACHABLE, TASK_DROPPED,
-      // TASK_GONE, TASK_GONE_BY_OPERATOR, and TASK_UNKNOWN.
+      // TASK_GONE, TASK_GONE_BY_OPERATOR, and TASK_UNKNOWN, and (2)
+      // the framework will assume responsibility for managing
+      // partitioned tasks that reregister with the master.
       //
-      // With this capability, frameworks can define how they would
-      // like to handle partitioned tasks. Frameworks will receive
-      // TASK_UNREACHABLE for tasks on partitioned agents; if/when the
-      // partitioned agent reregisters, the task will not be killed.
-      // Frameworks that enable this capability will never receive
-      // TASK_LOST; they will receive one of the most specific task
-      // statuses listed above instead.
+      // Frameworks that enable this capability can define how they
+      // would like to handle partitioned tasks. Frameworks will
+      // receive TASK_UNREACHABLE for tasks on agents that are
+      // partitioned from the master. If/when a partitioned agent
+      // reregisters, tasks on the agent that were started by
+      // PARTITION_AWARE frameworks will not killed.
       //
       // Without this capability, frameworks will receive TASK_LOST
       // for tasks on partitioned agents; such tasks will be killed by
@@ -1516,8 +1517,8 @@ enum TaskState {
   TASK_KILLED = 4;   // TERMINAL: The task was killed by the executor.
   TASK_ERROR = 7;    // TERMINAL: The task description contains an error.
 
-  // This is only sent when the framework does NOT opt-in to the
-  // PARTITION_AWARE capability.
+  // In Mesos 1.2, this will only be sent when the framework does NOT
+  // opt-in to the PARTITION_AWARE capability.
   TASK_LOST = 5;     // TERMINAL: The task failed but can be rescheduled.
 
   // The following task statuses are only sent when the framework
@@ -1526,8 +1527,8 @@ enum TaskState {
   // The task failed to launch because of a transient error. The
   // task's executor never started running. Unlike TASK_ERROR, the
   // task description is valid -- attempting to launch the task again
-  // may be successful. This is a terminal state.
-  TASK_DROPPED = 9;
+  // may be successful.
+  TASK_DROPPED = 9;  // TERMINAL.
 
   // The task was running on an agent that has lost contact with the
   // master, typically due to a network failure or partition. The task
@@ -1537,23 +1538,22 @@ enum TaskState {
   // The task was running on an agent that has been shutdown (e.g.,
   // the agent become partitioned, rebooted, and then reconnected to
   // the master; any tasks running before the reboot will transition
-  // from UNREACHABLE to GONE). The task is no longer running. This is
-  // a terminal state.
-  TASK_GONE = 11;
+  // from UNREACHABLE to GONE). The task is no longer running.
+  TASK_GONE = 11;    // TERMINAL.
 
   // The task was running on an agent that the master cannot contact;
   // the operator has asserted that the agent has been shutdown, but
   // this has not been directly confirmed by the master. If the
   // operator is correct, the task is not running and this is a
-  // terminal state; if the operator is mistaken, the task might still
-  // be running, and might return to the RUNNING state in the future.
+  // terminal state; if the operator is mistaken, the task may still
+  // be running and might return to RUNNING in the future.
   TASK_GONE_BY_OPERATOR = 12;
 
   // The master has no knowledge of the task. This is typically
   // because either (a) the master never had knowledge of the task, or
-  // (b) the master forgot about the task because it garbaged
-  // collected its metadata about the task. The task may or may not
-  // still be running.
+  // (b) the master forgot about the task because it garbage collected
+  // its metadata about the task. The task may or may not still be
+  // running.
   TASK_UNKNOWN = 13;
 }
 


[3/3] mesos git commit: Synchronized unversioned and v1 mesos.proto.

Posted by vi...@apache.org.
Synchronized unversioned and v1 mesos.proto.

Review: https://reviews.apache.org/r/52634/


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/0f97117b
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/0f97117b
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/0f97117b

Branch: refs/heads/master
Commit: 0f97117bac3e1382744e9a847ce11b7589fc45bd
Parents: 6c69a3c
Author: Neil Conway <ne...@gmail.com>
Authored: Fri Oct 7 12:38:38 2016 -0700
Committer: Vinod Kone <vi...@gmail.com>
Committed: Fri Oct 7 12:38:38 2016 -0700

----------------------------------------------------------------------
 include/mesos/v1/mesos.proto | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/0f97117b/include/mesos/v1/mesos.proto
----------------------------------------------------------------------
diff --git a/include/mesos/v1/mesos.proto b/include/mesos/v1/mesos.proto
index 212321b..301d1ae 100644
--- a/include/mesos/v1/mesos.proto
+++ b/include/mesos/v1/mesos.proto
@@ -1999,8 +1999,8 @@ message NetworkInfo {
 
 
 /**
- *  Encapsulation of `Capabilities` supported by Linux.
- *  Reference: http://linux.die.net/man/7/capabilities.
+ * Encapsulation of `Capabilities` supported by Linux.
+ * Reference: http://linux.die.net/man/7/capabilities.
  */
 message CapabilityInfo {
   // We start the actual values at an offset(1000) because Protobuf 2
@@ -2149,7 +2149,7 @@ message ContainerStatus {
   // Information about Linux control group (cgroup).
   optional CgroupInfo cgroup_info = 2;
 
-  //Information about Executor PID.
+  // Information about Executor PID.
   optional uint32 executor_pid = 3;
 }
 


[2/3] mesos git commit: Fixed typo in comment.

Posted by vi...@apache.org.
Fixed typo in comment.

Review: https://reviews.apache.org/r/52633/


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/6c69a3c8
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/6c69a3c8
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/6c69a3c8

Branch: refs/heads/master
Commit: 6c69a3c8065c8ddb946a8cdad9aa4258ab263225
Parents: f5b6a38
Author: Neil Conway <ne...@gmail.com>
Authored: Fri Oct 7 12:38:34 2016 -0700
Committer: Vinod Kone <vi...@gmail.com>
Committed: Fri Oct 7 12:38:34 2016 -0700

----------------------------------------------------------------------
 src/java/jni/org_apache_mesos_state_AbstractState.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/6c69a3c8/src/java/jni/org_apache_mesos_state_AbstractState.cpp
----------------------------------------------------------------------
diff --git a/src/java/jni/org_apache_mesos_state_AbstractState.cpp b/src/java/jni/org_apache_mesos_state_AbstractState.cpp
index 8addd2d..0f12f4f 100644
--- a/src/java/jni/org_apache_mesos_state_AbstractState.cpp
+++ b/src/java/jni/org_apache_mesos_state_AbstractState.cpp
@@ -47,7 +47,7 @@ extern "C" {
 // anonymous futures that used these function accidentally captured the 'thiz'
 // for the 'AbstractState' class, which was not intended and caused the C++
 // Future to be prematurely deleted as a result of the anonymous Future getting
-// garbaged collected by the JVM and invoking the finalizer which deleted the
+// garbage collected by the JVM and invoking the finalizer which deleted the
 // C++ Future. The intent was to capture the 'thiz' for the 'Future'. This is
 // done correctly by using the named inner classes.