[
https://issues.apache.org/jira/browse/YARN-4162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14949871#comment-14949871
]
Naganarasimha G R commented on YARN-4162:
-----------------------------------------
Hi [~wangda],
Thanks for the review comments and supporting with local testing.
bq. Is it possible to use the same PartitionQueueCapacitiesInfo instead of
check if csqinfo.label == null or not?
well i can avoid the if block but csqinfo.label itself cannot be set to the
default Partition as its also been used as flag to determine to show the leaf
queue in the normal way or the partition way.
bq. Why this isExclusiveNodeLabel check is needed?
isExclusiveNodeLabel is the check we had earlier in
CapacitySchedulerInfo.getQueues, basically to avoid displaying the queues which
is not accessible to a given NodeLabelPartition.
{code}
93 for (CSQueue queue : parentQueue.getChildQueues()) { 92
for (CSQueue queue : parentQueue.getChildQueues()) {
94 if (nodeLabel.getIsExclusive()
95 && !((AbstractCSQueue) queue).accessibleToPartition(nodeLabel
96 .getLabelName())) {
97 // Skip displaying the hierarchy for the queues for which the
exclusive
98 // labels are not accessible
99 continue;
100 }
{code}
bq. Could you update <DEFAULT_PARTITION> to DEFAULT_PARTITION?
Well shall i update in all places displayed in UI or only in REST ?
For other comments will get it updated in the next patch .
> Scheduler info in REST, is currently not displaying partition specific queue
> information similar to UI
> ------------------------------------------------------------------------------------------------------
>
> Key: YARN-4162
> URL: https://issues.apache.org/jira/browse/YARN-4162
> Project: Hadoop YARN
> Issue Type: Sub-task
> Components: api, client, resourcemanager
> Reporter: Naganarasimha G R
> Assignee: Naganarasimha G R
> Attachments: YARN-4162.v1.001.patch, YARN-4162.v2.001.patch,
> YARN-4162.v2.002.patch, YARN-4162.v2.003.patch, restAndJsonOutput.zip
>
>
> When Node Labels are enabled then REST Scheduler Information should also
> provide partition specific queue information similar to the existing Web UI
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
