[jira] [Commented] (TEZ-3857) Tez TaskImpl can throw Invalid state transition for leaf tasks that do Retro Active Transition

["Jason Lowe (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/TEZ-3857?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16234209#comment-16234209 ] 

Jason Lowe commented on TEZ-3857:
---------------------------------

Thanks for updating the patch!

+1 lgtm.


> Tez TaskImpl can throw Invalid state transition for leaf tasks that do Retro Active Transition
> ----------------------------------------------------------------------------------------------
>
>                 Key: TEZ-3857
>                 URL: https://issues.apache.org/jira/browse/TEZ-3857
>             Project: Apache Tez
>          Issue Type: Bug
>            Reporter: Kuhu Shukla
>            Assignee: Kuhu Shukla
>            Priority: Major
>         Attachments: TEZ-3857.001.patch, TEZ-3857.002.patch, TEZ-3857.003.patch
>
>
> {code}
> Invalid event T_ATTEMPT_FAILED on Task task_1234_5678_1_01_000001
> {code}
> The task had more than one running attempts (because of speculative execution), while one of them succeeded and the task was marked succeeded, the second failed and caused the Task state machine to enter error state since the task was in a leaf vertex and does the following:
> {code}
> if (task.leafVertex) {
>         LOG.error("Unexpected event for task of leaf vertex " + event.getType() + ", taskId: "
>             + task.getTaskId());
>         task.internalError(event.getType());
>       }
> {code}
> This JIRA tracks fixing this invalid state.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)