[
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)
