[jira] [Commented] (TEZ-657) Tez should process the Container exit status - specifically when the RM preempts a container

["Siddharth Seth (JIRA)" <ji...@apache.org>]

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

Siddharth Seth commented on TEZ-657:
------------------------------------

Looks good.
Minor - I think it's better to localize all the mocks within a unit test instead of at the class level. Makes addition of new unit tests which may need different mocks easier. IAC, this can be changed at a later point if required.

> Tez should process the Container exit status - specifically when the RM preempts a container
> --------------------------------------------------------------------------------------------
>
>                 Key: TEZ-657
>                 URL: https://issues.apache.org/jira/browse/TEZ-657
>             Project: Apache Tez
>          Issue Type: Bug
>            Reporter: Siddharth Seth
>            Assignee: Bikas Saha
>         Attachments: TEZ-657.1.patch
>
>
> Containers preempted by the RM will currently register as task failures - these tasks should be considered to be KILLED instead.
> Handling the entire preemption hint logic would be a separate jira.



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