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

Hitesh Shah commented on TEZ-1642:
----------------------------------

Patch looks fine for the most part. 

How is printHistoryEvents() meant to be help someone trying to debug a test 
failure? It has no information on the first and last events and also there is 
no additional log saying parsing events from log1 or log2. Also, it might be 
helpful to log a statement at the start of each test case so that one can 
easily figure out which events were generated as part of which test case. 



> TestAMRecovery sometimes fail
> -----------------------------
>
>                 Key: TEZ-1642
>                 URL: https://issues.apache.org/jira/browse/TEZ-1642
>             Project: Apache Tez
>          Issue Type: Bug
>            Reporter: Jeff Zhang
>            Assignee: Jeff Zhang
>         Attachments: TEZ-1642.patch
>
>




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

Reply via email to