Wangda Tan commented on YARN-4304:

Hi [~sunilg].

1) CSQueueUtils calculate max available resource to queue may not correct to me:
In my mind, it should be: 
available-resource = Σ (componentwiseMax(p.max-resource - p.used, 0))
                     p ∈ {accessible-partitions-of-queue}
We should only consider accessible-partitions, and if a partition's usage more 
than max-resource, we should consider it's 0 instead of a negative value.

2) Looked at REST/web API changes as well, seems good to me. Could you attach 
screenshot/REST-response as well?

> AM max resource configuration per partition to be displayed/updated correctly 
> in UI and in various partition related metrics
> ----------------------------------------------------------------------------------------------------------------------------
>                 Key: YARN-4304
>                 URL: https://issues.apache.org/jira/browse/YARN-4304
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: webapp
>    Affects Versions: 2.7.1
>            Reporter: Sunil G
>            Assignee: Sunil G
>         Attachments: 0001-YARN-4304.patch
> As we are supporting per-partition level max AM resource percentage 
> configuration, UI and various metrics also need to display correct 
> configurations related to same. 
> For eg: Current UI still shows am-resource percentage per queue level. This 
> is to be updated correctly when label config is used.
> - Display max-am-percentage per-partition in Scheduler UI (label also) and in 
> ClusterMetrics page
> - Update queue/partition related metrics w.r.t per-partition 
> am-resource-percentage

This message was sent by Atlassian JIRA

Reply via email to