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

Jeff Zhang commented on TEZ-2579:
---------------------------------

Committed to 0.6, 0.7 & master

> Incorrect comparison of TaskAttemptId
> -------------------------------------
>
>                 Key: TEZ-2579
>                 URL: https://issues.apache.org/jira/browse/TEZ-2579
>             Project: Apache Tez
>          Issue Type: Bug
>            Reporter: Jeff Zhang
>            Assignee: Jeff Zhang
>             Fix For: 0.6.2, 0.8.0, 0.7.1
>
>         Attachments: TEZ-2579-1.patch, TEZ-2579-2.patch
>
>
> TaskImpl#AttemptSucceededTransition
> {code}
>       // issue kill to all other attempts
>       for (TaskAttempt attempt : task.attempts.values()) {
>         if (attempt.getID() != task.successfulAttempt &&  // should use 
> !equals 
>         !attempt.isFinished()) {           // but it won't affect the state 
> machine transition, because the successful task attempt should already 
> complete. 
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to