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

Alejandro Abdelnur commented on YARN-937:
-----------------------------------------

bq. Since the AMRMToken already has the service field populated we dont need to 
override anything. So we dont need to lookup any address from config in the 
YARNClient code. Later, for HA if we need to do some translation, then it 
should probably happen via the RMProxy layer. Does that work for you?

Well, [~daryn] has been doing lot of work to ensure the service of a token is 
not set by the server but by the client. Doing what you suggest is going 
opposite to that.

This has to be done in the client (for example in the case of a multi-homing 
setup, it would not work otherwise as the RM does not know the hostname/IP 
visible to the user).

Also, looking at the conf is exactly what the {{ClientRMProxy}} is doing within 
the {{getRMAddress()}}.

I think the current patch it is the right approach.


                
> Fix unmanaged AM in non-secure/secure setup post YARN-701
> ---------------------------------------------------------
>
>                 Key: YARN-937
>                 URL: https://issues.apache.org/jira/browse/YARN-937
>             Project: Hadoop YARN
>          Issue Type: Bug
>    Affects Versions: 2.1.0-beta
>            Reporter: Arun C Murthy
>            Assignee: Alejandro Abdelnur
>            Priority: Blocker
>             Fix For: 2.1.0-beta
>
>         Attachments: YARN-937.patch, YARN-937.patch, YARN-937.patch, 
> YARN-937.patch, YARN-937.patch, YARN-937.patch
>
>
> Fix unmanaged AM in non-secure/secure setup post YARN-701 since app-tokens 
> will be used in both scenarios.

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