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)