Update cgroups documentation - was missing a necessary yarn-site configuration.
Fix minor typo hierachy->hierarchy.
Pull Request:
Closes #65
Author: Darin Johnson <darinj@apache.org>
diff --git a/docs/cgroups.md b/docs/cgroups.md
index d28eeab..b90c781 100644
--- a/docs/cgroups.md
+++ b/docs/cgroups.md
@@ -82,6 +82,10 @@
<name>yarn.nodemanager.linux-container-executor.path</name>
<value>${yarn.home}/bin/container-executor</value>
</property>
+<property>
+<name>yarn.nodemanager.linux-container-executor.cgroups.hierarchy</name>
+<value>${yarn.nodemanager.linux-container-executor.cgroups.hierarchy}</value>
+</property>
<!-- Optional parameters, usually unnecessary
<property>
diff --git a/myriad-scheduler/src/main/java/org/apache/myriad/scheduler/NMExecutorCLGenImpl.java b/myriad-scheduler/src/main/java/org/apache/myriad/scheduler/NMExecutorCLGenImpl.java
index 8fca9e2..18c3fce 100644
--- a/myriad-scheduler/src/main/java/org/apache/myriad/scheduler/NMExecutorCLGenImpl.java
+++ b/myriad-scheduler/src/main/java/org/apache/myriad/scheduler/NMExecutorCLGenImpl.java
@@ -41,7 +41,7 @@
* YARN class to help handle LCE resources
*/
// TODO (mohit): Should it be configurable ?
- public static final String KEY_YARN_NM_LCE_CGROUPS_HIERARCHY = "yarn.nodemanager.linux-container-executor.cgroups.hierachy";
+ public static final String KEY_YARN_NM_LCE_CGROUPS_HIERARCHY = "yarn.nodemanager.linux-container-executor.cgroups.hierarchy";
public static final String KEY_YARN_HOME = "yarn.home";
public static final String KEY_NM_RESOURCE_CPU_VCORES = "nodemanager.resource.cpu-vcores";
public static final String KEY_NM_RESOURCE_MEM_MB = "nodemanager.resource.memory-mb";