[ 
https://issues.apache.org/jira/browse/YARN-3671?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Subru Krishnan updated YARN-3671:
---------------------------------
    Attachment: YARN-3671-YARN-2915-v2.patch

Thanks [~jianhe] for the feedback. Updated patch (v2) to remove redundant null 
check and refactor _setStateStoreClient_ as suggested by you.

As to your other questions:

bq. we already have RM_CLUSTER_ID, any chance that this can be used for 
FEDERATION_SUBCLUSTER_ID ?

That's a possibility. The reason I didn't combine both is RM_CLUSTER_ID is 
currently used for HA but Federation can work both with and without HA (and RM 
HA can work both with and without Federation). So felt it would be better to 
keep them separate. Thoughts?

bq. I feel the SubClusterState is a bit redundant in the request object, 
because the API itself already indicates the state such as register / 
deregister.

You are right. We don't want state to be null in the store so either the store 
impl can implicitly add SC_NEW/SC_UNREGISTERED on register / deregister or the 
invoker (which is always RM) can. I decided to do it in the RM for 2 reasons:
  1. It is trivial (one line) & needs to be done in a single place (RM) instead 
of in each store impl we add.
   2. This allows for flexibility future as RM could potentially register / 
deregister with different states (say SC_DRAINING).

Makes sense?

> Integrate Federation services with ResourceManager
> --------------------------------------------------
>
>                 Key: YARN-3671
>                 URL: https://issues.apache.org/jira/browse/YARN-3671
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: nodemanager, resourcemanager
>            Reporter: Subru Krishnan
>            Assignee: Subru Krishnan
>         Attachments: YARN-3671-YARN-2915-v1.patch, 
> YARN-3671-YARN-2915-v2.patch
>
>
> This JIRA proposes adding the ability to turn on Federation services like 
> StateStore, cluster membership heartbeat etc in the RM



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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

Reply via email to