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

Szilard Nemeth commented on YARN-9092:
--------------------------------------

[~devaraj.k], [~tangzhankun], [~zyluo], [~sunilg], [~leftnoteasy]: As you have 
a lot expertise in this area, could you please help me to review (and hopefully 
commit) this? 

Thanks!

> Create an object for cgroups mount enable and cgroups mount path as they 
> belong together
> ----------------------------------------------------------------------------------------
>
>                 Key: YARN-9092
>                 URL: https://issues.apache.org/jira/browse/YARN-9092
>             Project: Hadoop YARN
>          Issue Type: Improvement
>            Reporter: Szilard Nemeth
>            Assignee: Szilard Nemeth
>            Priority: Minor
>         Attachments: YARN-9092.001.patch, YARN-9092.002.patch, 
> YARN-9092.003.patch
>
>
> YarnConfiguration.NM_LINUX_CONTAINER_CGROUPS_MOUNT and 
> YarnConfiguration.NM_LINUX_CONTAINER_CGROUPS_MOUNT_PATH are used in 
> conjunction many places in the code, so for the sake of readabilty and 
> simplicity, it is better to wrap the values of these configs to an object and 
> use it instead of having 2 fields in 
> CGroupsHandlerImpl and in CgroupsLCEResourcesHandler as well.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to