[ 
https://issues.apache.org/jira/browse/MAPREDUCE-5562?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Zhijie Shen updated MAPREDUCE-5562:
-----------------------------------

    Attachment: MAPREDUCE-5562.5.patch

Created a patch according to Jason's suggestion. Basically we do the following 
things:

1. Move the flag "successfulUnregistration" (which used to be 
safeToReportTerminationToUser) to RMCommunicator#unregister, such that  
cleaning the staging dir is able to be aware of this flag as well

2. The unregister fails, before cleaning the staging dir, we revisit 
"isLastRetry" to see whether AM really doesn't have a chance or not. We only 
clean the dir when no more retries are allowed according to the configured 
maxAttempts.

3. Added the test case of cleaning the staging dir on unregister failure, and 
modified the tests which are related to refactoring successfulUnregistration.

> MR AM should exit when unregister() throws exception
> ----------------------------------------------------
>
>                 Key: MAPREDUCE-5562
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-5562
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>            Reporter: Zhijie Shen
>            Assignee: Zhijie Shen
>         Attachments: MAPREDUCE-5562.1.patch, MAPREDUCE-5562.2.patch, 
> MAPREDUCE-5562.3.patch, MAPREDUCE-5562.5.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.1#6144)

Reply via email to