[
https://issues.apache.org/jira/browse/YARN-6128?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16246410#comment-16246410
]
Subru Krishnan commented on YARN-6128:
--------------------------------------
Thanks [~botong] for addressing my comments, the latest patch is very close. I
have a few minor follow ups:
* Can you double check the {{Credentials}} don't include the original
_AMRMToken_ as that can potentially allow AMs to bypass _AMRMProxy_?
* Also we need to add {{Credentials}} only if HA/recovery is enabled. You'll
then have to null check before loading {{Credentials}} to UGI in
{{FederationInterptor}}.
* Please update the {{Federation.md}} documentation to include a description of
HA.
* Nit:there are couple of checkstyle warnings.
> Add support for AMRMProxy HA
> ----------------------------
>
> Key: YARN-6128
> URL: https://issues.apache.org/jira/browse/YARN-6128
> Project: Hadoop YARN
> Issue Type: Sub-task
> Components: amrmproxy, nodemanager
> Reporter: Subru Krishnan
> Assignee: Botong Huang
> Attachments: YARN-6128.v0.patch, YARN-6128.v1.patch,
> YARN-6128.v1.patch, YARN-6128.v2.patch, YARN-6128.v3.patch,
> YARN-6128.v3.patch, YARN-6128.v4.patch, YARN-6128.v5.patch,
> YARN-6128.v6.patch, YARN-6128.v7.patch
>
>
> YARN-556 added the ability for RM failover without loosing any running
> applications. In a Federated YARN environment, there's additional state in
> the {{AMRMProxy}} to allow for spanning across multiple sub-clusters, so we
> need to enhance {{AMRMProxy}} to support HA.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
