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

Jeff Zhang commented on TEZ-2752:
---------------------------------

bq. If the time for losing the output is needed (not sure if it is right now) 
then that can be tracked separately. 
This is what I concern. Let's add that in future if it is necessary. +1

> logUnsuccessful completion in Attempt should write original finish time to ATS
> ------------------------------------------------------------------------------
>
>                 Key: TEZ-2752
>                 URL: https://issues.apache.org/jira/browse/TEZ-2752
>             Project: Apache Tez
>          Issue Type: Bug
>            Reporter: Bikas Saha
>            Assignee: Bikas Saha
>         Attachments: TEZ-2752.1.patch
>
>
> Currently it overwrites with the current time and that results in wrong 
> information since in reality the attempt might have finished earlier if the 
> failure was output failure. Other failures also set the finish time in the 
> terminate transition and thus even in non output failed cases the value of 
> finishtime can be used instead of using current time.



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

Reply via email to