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

ASF GitHub Bot commented on YARN-11491:
---------------------------------------

goiri commented on code in PR #5637:
URL: https://github.com/apache/hadoop/pull/5637#discussion_r1261674724


##########
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/conf/YarnConfiguration.java:
##########
@@ -4001,7 +4001,7 @@ public static boolean isAclEnabled(Configuration conf) {
       FEDERATION_PREFIX + "state-store.class";
 
   public static final String DEFAULT_FEDERATION_STATESTORE_CLIENT_CLASS =
-      
"org.apache.hadoop.yarn.server.federation.store.impl.MemoryFederationStateStore";
+      
"org.apache.hadoop.yarn.server.federation.store.impl.ZookeeperFederationStateStore";

Review Comment:
   Do we have this also in some of the XMLs?
   The other thing is we could use the getClass() and getName()





> [Federation] Use ZookeeperFederationStateStore as the DefaultStateStore
> -----------------------------------------------------------------------
>
>                 Key: YARN-11491
>                 URL: https://issues.apache.org/jira/browse/YARN-11491
>             Project: Hadoop YARN
>          Issue Type: Improvement
>          Components: federation
>    Affects Versions: 3.4.0
>            Reporter: Shilun Fan
>            Assignee: Shilun Fan
>            Priority: Major
>              Labels: pull-request-available
>
> We currently use MemoryStateStore as the default StateStore, in the 
> production environment, we should use ZookeeperFederationStateStore as the 
> default StateStore.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

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