Yan Xu created MESOS-8855:
-----------------------------

             Summary: Change TaskStatus.Reason's default value to something 
                 Key: MESOS-8855
                 URL: https://issues.apache.org/jira/browse/MESOS-8855
             Project: Mesos
          Issue Type: Bug
            Reporter: Yan Xu


We are constantly adding new task reasons and they'll result in the default 
enum value on clients that don't recognize them, right now the default value 
(first value) is {{REASON_COMMAND_EXECUTOR_FAILED}} and we should change it to 
something more legit. 

Also [~jieyu] has this TODO

{code:title=}
enum Reason {
    // TODO(jieyu): The default value when a caller doesn't check for
    // presence is 0 and so ideally the 0 reason is not a valid one.
    // Since this is not used anywhere, consider removing this reason.
    REASON_COMMAND_EXECUTOR_FAILED = 0;
}
{code}

Note that Mesos already defines an used {{REASON_TASK_UNKNOWN}} and the fact 
that there's task state {{TASK_UNKNOWN}} may influence the naming of the 
default enum field.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to