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

Hitesh Shah commented on TEZ-1325:
----------------------------------

Comments: 

 - add VisibleForTest to DAGSummaryData
 - "/tmp/app1/recovery" should not be used as a hardcoded path. Better to 
reference something under target. Look at TestTezJobs for example. Or use a 
mock Path instance if this is not meant to be used. 



> RecoveryParser can find incorrect last DAG ID
> ---------------------------------------------
>
>                 Key: TEZ-1325
>                 URL: https://issues.apache.org/jira/browse/TEZ-1325
>             Project: Apache Tez
>          Issue Type: Sub-task
>    Affects Versions: 0.4.0
>            Reporter: Jeff Zhang
>            Assignee: Jeff Zhang
>         Attachments: Tez-1325-2.patch, Tez-1325.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to