[ https://issues.apache.org/jira/browse/MESOS-7975?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16169475#comment-16169475 ]
Qian Zhang commented on MESOS-7975: ----------------------------------- [~alexr] Yeah, in mesos.proto, for {{TASK_FINISHED}}, I only see a comment {{// TERMINAL: The task finished successfully}} which seems not very clear, different people may have different understanding for that. We may need to send a mail to dev & user list to let everyone know our proposal and collect feedbacks, and eventually reach a consensus. > The command/default executor can incorrectly send a TASK_FINISHED update even > when the task is killed > ----------------------------------------------------------------------------------------------------- > > Key: MESOS-7975 > URL: https://issues.apache.org/jira/browse/MESOS-7975 > Project: Mesos > Issue Type: Bug > Reporter: Anand Mazumdar > Assignee: Qian Zhang > Priority: Critical > Labels: mesosphere > > Currently, when a task is killed, the default and the command executor > incorrectly send a {{TASK_FINISHED}} status update instead of > {{TASK_KILLED}}. This is due to an unfortunate missed conditional check when > the task exits with a zero status code. > {code} > if (WSUCCEEDED(status)) { > taskState = TASK_FINISHED; > } else if (killed) { > // Send TASK_KILLED if the task was killed as a result of > // kill() or shutdown(). > taskState = TASK_KILLED; > } else { > taskState = TASK_FAILED; > } > {code} > We should modify the code to correctly send {{TASK_KILLED}} status updates > when a task is killed. -- This message was sent by Atlassian JIRA (v6.4.14#64029)