[ https://issues.apache.org/jira/browse/YARN-1781?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13921804#comment-13921804 ]
Xuan Gong commented on YARN-1781: --------------------------------- The patch looks fine to me. One nit: {code} + diskUtilizationPercCutoff = utilizationPercCutOff; + diskUtilizationSpaceCutoff = utilizationSpaceCutOff; + diskUtilizationPercCutoff = + utilizationPercCutOff < 0.0F ? 0.0F : utilizationPercCutOff; + diskUtilizationPercCutoff = + utilizationPercCutOff > 100.0F ? 100.0F : diskUtilizationPercCutoff; + diskUtilizationSpaceCutoff = + utilizationSpaceCutOff < 0 ? 0 : utilizationSpaceCutOff; {code} Maybe we can simplify a little bit. {code} diskUtilizationPercCutoff = utilizationPercCutOff < 0.0F ? 0.0F : utilizationPercCutOff > 100.0F ? 100.0F : utilizationPercCutOff; diskUtilizationSpaceCutoff = utilizationSpaceCutOff < 0 ? 0 : utilizationSpaceCutOff; {code} > NM should allow users to specify max disk utilization for local disks > --------------------------------------------------------------------- > > Key: YARN-1781 > URL: https://issues.apache.org/jira/browse/YARN-1781 > Project: Hadoop YARN > Issue Type: Sub-task > Components: nodemanager > Reporter: Varun Vasudev > Assignee: Varun Vasudev > Attachments: apache-yarn-1781.0.patch, apache-yarn-1781.1.patch > > > This is related to YARN-257(it's probably a sub task?). Currently, the NM > does not detect full disks and allows full disks to be used by containers > leading to repeated failures. YARN-257 deals with graceful handling of full > disks. This ticket is only about detection of full disks by the disk health > checkers. > The NM should allow users to set a maximum disk utilization for local disks > and mark disks as bad once they exceed that utilization. At the very least, > the NM should at least detect full disks. -- This message was sent by Atlassian JIRA (v6.2#6252)