You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@tez.apache.org by "Hitesh Shah (JIRA)" <ji...@apache.org> on 2014/09/09 20:39:29 UTC

[jira] [Commented] (TEZ-850) Recovery unit tests

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

Hitesh Shah commented on TEZ-850:
---------------------------------

Mostly looks fine. 

Couple of minor comments:
   - for fields changed from private to package public - please add a VisibleForTesting annotation
   - Also, for certain critical events that are logged out of band, we need some additional tests to test scenarios where the out of band event was logged in summary file but the main event was not logged in dag recovery file. This could be done as a separate jira.  

> Recovery unit tests
> -------------------
>
>                 Key: TEZ-850
>                 URL: https://issues.apache.org/jira/browse/TEZ-850
>             Project: Apache Tez
>          Issue Type: Sub-task
>            Reporter: Hitesh Shah
>            Assignee: Jeff Zhang
>         Attachments: Tez-850-2.patch, Tez-850.patch
>
>
> Tests for custom edge managers, groups handling, etc.



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