hadoop-yarn-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Subru Krishnan (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (YARN-6128) Add support for AMRMProxy HA
Date Thu, 09 Nov 2017 20:03:00 GMT

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

To unsubscribe, e-mail: yarn-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: yarn-issues-help@hadoop.apache.org

View raw message