[ 
https://issues.apache.org/jira/browse/YARN-6909?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sunil G updated YARN-6909:
--------------------------
    Attachment: YARN-6909.005.patch

Thanks [~templedf]. Fixed all comments, except below.
bq.Shouldn't both the newDefaultInformation() be methods belong to the 
ResourceInformation class?
We have seen a bit for performance dip when using same api from 
ResourceInformation. I will again check and see how far is off now.

> The performance advantages of YARN-6679 are lost when resource types are used
> -----------------------------------------------------------------------------
>
>                 Key: YARN-6909
>                 URL: https://issues.apache.org/jira/browse/YARN-6909
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: resourcemanager
>    Affects Versions: YARN-3926
>            Reporter: Daniel Templeton
>            Assignee: Sunil G
>            Priority: Critical
>         Attachments: YARN-6909.001.patch, YARN-6909.002.patch, 
> YARN-6909.003.patch, YARN-6909.004.patch, YARN-6909.005.patch
>
>
> YARN-6679 added the {{SimpleResource}} as a lightweight replacement for 
> {{ResourcePBImpl}} when a protobuf isn't needed.  With resource types enabled 
> and anything other than memory and CPU defined, {{ResourcePBImpl}} will 
> always be used.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
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