[
https://issues.apache.org/jira/browse/YARN-11393?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17651198#comment-17651198
]
ASF GitHub Bot commented on YARN-11393:
---------------------------------------
bteke commented on code in PR #5201:
URL: https://github.com/apache/hadoop/pull/5201#discussion_r1055268220
##########
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/fair/converter/TestFSConfigToCSConfigConverter.java:
##########
@@ -185,6 +186,20 @@ public void testDefaultMaxAMShare() throws Exception {
conf.get(PREFIX + "root.users.joe maximum-am-resource-percent"));
}
+ @Test
+ public void testDefaultUserLimitFactor() throws Exception {
+ converter.convert(config);
+
+ Configuration conf = converter.getCapacitySchedulerConfig();
+ String userLimitFactor =
+ conf.get(PREFIX + "root.default." + USER_LIMIT_FACTOR);
+
+ assertEquals("Default user limit factor", "-1.0", userLimitFactor);
+
+ assertNull("root.users.joe user-limit-factor should be null",
+ conf.get(PREFIX + "root.users.joe user-limit-factor"));
Review Comment:
This should be -1 as well (if root.users.joe is an existing static queue).
##########
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/fair/converter/FSConfigToCSConfigConverter.java:
##########
@@ -412,6 +414,14 @@ private void emitDefaultMaxAMShare() {
queueMaxAMShareDefault);
}
}
+
+ private void emitDefaultUserLimitFactor() {
+ capacitySchedulerConfig.setFloat(
+ CapacitySchedulerConfiguration.
+ PREFIX + "root.default." + USER_LIMIT_FACTOR,
Review Comment:
There might be other queues besides the default one which also need to have
the default user-limit-factor. Can you please address that?
> Fs2cs could be extended to set ULF to -1 upon conversion
> --------------------------------------------------------
>
> Key: YARN-11393
> URL: https://issues.apache.org/jira/browse/YARN-11393
> Project: Hadoop YARN
> Issue Type: Improvement
> Components: yarn
> Reporter: Susheel Gupta
> Assignee: Susheel Gupta
> Priority: Major
> Labels: pull-request-available
>
> A global configuration to set the default User Limit Factor to -1 on newly
> created queues.
> To solve this is to make fs2cs (Fair Scheduler to Capacity Scheduler tool)
> add the user-limit-factor value -1 to the conversion as default.
--
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
