p-szucs commented on code in PR #5332: URL: https://github.com/apache/hadoop/pull/5332#discussion_r1232472683
########## hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/fair/converter/QueuePlacementConverter.java: ########## @@ -187,7 +188,7 @@ private Rule createNestedRule(Policy policy, boolean create, FSConfigToCSConfigRuleHandler ruleHandler, boolean fsParentCreate, - String parentQueue, + QueuePath parentQueue, Review Comment: As I can see it should be the full queue path. The queue name gets its value from the defaultQueueName member of the DefaultPlacementRule class: https://github.com/apache/hadoop/blob/1f73332bb6632358b8f8272c8c8a8b617a0c6cb0/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/fair/converter/QueuePlacementConverter.java#L134 In the DefaultPlacementRule class we call an assureRoot() method on it before setting it from the configuration, like this: https://github.com/apache/hadoop/blob/1f73332bb6632358b8f8272c8c8a8b617a0c6cb0/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/placement/DefaultPlacementRule.java#L74 If I'm not mistaken with this, it should ensure that we are using the full queue path here as well. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org