You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@mesos.apache.org by "Yan Xu (JIRA)" <ji...@apache.org> on 2018/04/30 18:32:00 UTC

[jira] [Created] (MESOS-8855) Change TaskStatus.Reason's default value to something

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)