[ 
https://issues.apache.org/jira/browse/YARN-6578?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16477463#comment-16477463
 ] 

Naganarasimha G R commented on YARN-6578:
-----------------------------------------

[~cheersyang] & [~fly_in_gis], 

I went through the approach, but was wondering its not covering the scenario 
with resource profiles, if multiple resource types are supported then this 
would be like not a proper solution in terms of the Object 
{{ResourceUtilization}} being returned & changes in{{ContainerStatusPBImpl}} by 
storing only .

Was wondering whether we need to return {{ResourceInformation }}and store 
resource type information and individual resource values as separate key value 
pairs in the {{ContainerStatusPBImpl.containerAttributes}}  map ?

cc [~sunil.gov...@gmail.com]

> Return container resource utilization from NM ContainerStatus call
> ------------------------------------------------------------------
>
>                 Key: YARN-6578
>                 URL: https://issues.apache.org/jira/browse/YARN-6578
>             Project: Hadoop YARN
>          Issue Type: Improvement
>            Reporter: Yang Wang
>            Assignee: Yang Wang
>            Priority: Major
>         Attachments: YARN-6578.001.patch, YARN-6578.002.patch, 
> YARN-6578.003.patch
>
>
> When the applicationMaster wants to change(increase/decrease) resources of an 
> allocated container, resource utilization is an important reference indicator 
> for decision making. So, when AM call NMClient.getContainerStatus, resource 
> utilization needs to be returned.
> Also container resource utilization need to report to RM to make better 
> scheduling.
> So put resource utilization in ContainerStatus.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to