Varun Vasudev updated YARN-3542:
    Attachment: YARN-3542.006.patch

bq. Adding a @Deprecated to TestCgroupsLCEResourcesHandler would remove the 
deprecated warning use - this test class should be deprecated anyway?

Yep. Fixed.

bq. Some checkstyle comments need looking into (line lengths for new code, 

Fixed as many as I could - the rest are package lines and one config variable.

> Re-factor support for CPU as a resource using the new ResourceHandler 
> mechanism
> -------------------------------------------------------------------------------
>                 Key: YARN-3542
>                 URL: https://issues.apache.org/jira/browse/YARN-3542
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: nodemanager
>            Reporter: Sidharta Seethana
>            Assignee: Varun Vasudev
>            Priority: Critical
>         Attachments: YARN-3542.001.patch, YARN-3542.002.patch, 
> YARN-3542.003.patch, YARN-3542.004.patch, YARN-3542.005.patch, 
> YARN-3542.006.patch
> In YARN-3443 , a new ResourceHandler mechanism was added which enabled easier 
> addition of new resource types in the nodemanager (this was used for network 
> as a resource - See YARN-2140 ). We should refactor the existing CPU 
> implementation ( LinuxContainerExecutor/CgroupsLCEResourcesHandler ) using 
> the new ResourceHandler mechanism. 

This message was sent by Atlassian JIRA

Reply via email to