[
https://issues.apache.org/jira/browse/MAPREDUCE-5562?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13787344#comment-13787344
]
Zhijie Shen commented on MAPREDUCE-5562:
----------------------------------------
bq. Always return the final state if this is the configuration last retry , so
that we don't need to query RM in the end?
Yes, it can prevent Client from querying RM. However, this brings a logic
error. Before unregistration, successfullyUnregistered = false. If it happens
to be the final attempts, isLastAMRetry = true is set in serviceInit().
Therefore, before unregistration, client will get the final state, which is
originally what we don't what. Therefore, it's a compromise that returning
RUNNING to the client when unregistration fails and it's the last retry.
Anyway, the client will see the correct state from RM. Moreover, it's a even
more are case, such that RM should not expect many queries like this.
> 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, MAPREDUCE-5562.6.patch
>
>
--
This message was sent by Atlassian JIRA
(v6.1#6144)
