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

Bikas Saha commented on MAPREDUCE-5505:
---------------------------------------

I dont think making jobimpl aware of all this is a good idea. MRClientService 
is already the last thing to stop. So its a good place to contain this race 
condition handling logic. We will probably need to move the getReport() method 
from JobImpl -> MRClientService so that the translation happens over there.
However, if this needs to be urgently fixed and other folks think this is a 
good idea then I am ok with this approach.
                
> Clients should be notified job finished only after job successfully 
> unregistered 
> ---------------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-5505
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-5505
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>            Reporter: Jian He
>            Assignee: Zhijie Shen
>         Attachments: MAPREDUCE-5505.1.patch, MAPREDUCE-5505.1.patch
>
>
> This is to make sure user is notified job finished after job is really done. 
> This does increase client latency but can reduce some races during unregister 
> like YARN-540

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to