[ https://issues.apache.org/jira/browse/YARN-6799?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Yufei Gu updated YARN-6799: --------------------------- Issue Type: Improvement (was: Bug) > Remove the duplicated code in CGroupsHandlerImp.java > ---------------------------------------------------- > > Key: YARN-6799 > URL: https://issues.apache.org/jira/browse/YARN-6799 > Project: Hadoop YARN > Issue Type: Improvement > Components: nodemanager > Affects Versions: 3.0.0-alpha3 > Reporter: Yufei Gu > Priority: Trivial > > The else clause in initializeCGroupController is not necessary. > {code} > public void initializeCGroupController(CGroupController controller) throws > ResourceHandlerException { > if (enableCGroupMount) { > // We have a controller that needs to be mounted > mountCGroupController(controller); > } else { > String controllerPath = getControllerPath(controller); > if (controllerPath == null) { > throw new ResourceHandlerException( > String.format("Controller %s not mounted." > + " You either need to mount it with %s" > + " or mount cgroups before launching Yarn", > controller.getName(), YarnConfiguration. > NM_LINUX_CONTAINER_CGROUPS_MOUNT)); > } > } > // We are working with a pre-mounted contoller > // Make sure that Yarn cgroup hierarchy path exists > initializePreMountedCGroupController(controller); > } > private void initializePreMountedCGroupController(CGroupController > controller) > throws ResourceHandlerException { > // Check permissions to cgroup hierarchy and > // create YARN cgroup if it does not exist, yet > String controllerPath = getControllerPath(controller); > if (controllerPath == null) { > throw new ResourceHandlerException( > String.format("Controller %s not mounted." > + " You either need to mount it with %s" > + " or mount cgroups before launching Yarn", > controller.getName(), YarnConfiguration. > NM_LINUX_CONTAINER_CGROUPS_MOUNT)); > } > {code} -- 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