[ https://issues.apache.org/jira/browse/YARN-4304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15027907#comment-15027907 ]
Naganarasimha G R commented on YARN-4304: ----------------------------------------- Thanks [~wangda] for sharing your thoughts, bq. Instead of putting max am to a separated object, I would prefer to put them to existing resourceUsageByPartition instead of introducing a new object. Even though max-am-limit is not usage, but it describes upper bound of usage. Thoughts? By {{resourceUsageByPartition}} you refer to {{ResourceUsageInfo}} present in the parent class {{CapacitySchedulerQueueInfo}}, if so yes and any way as these new classes have not gone into any version yet, so we can rename them as appropriately like {{resourceUsagesByPartition}} => {{resourceInfoByPartition}}, {{ResourceUsageInfo}} => {{ResourceInfo}} & {{PartitionResourceUsageInfo}} => {{PartitionResourceInfo}}. Thoughts ? > 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, 0002-YARN-4304.patch, > 0003-YARN-4304.patch, REST_and_UI.zip > > > 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 (v6.3.4#6332)