Repository: hadoop
Updated Branches:
  refs/heads/branch-2 d63e98ad5 -> 221c1bdf7


YARN-7099. ResourceHandlerModule.parseConfiguredCGroupPath only works for 
privileged yarn users. (Contributed by Miklos Szegedi via Yufei Gu)

(cherry picked from commit d5bc9582793de6ab2d2e489cde293bf076229a1c)


Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/221c1bdf
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/221c1bdf
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/221c1bdf

Branch: refs/heads/branch-2
Commit: 221c1bdf7e28f9db1efef409044e6e6b7062d3d8
Parents: d63e98a
Author: Yufei Gu <[email protected]>
Authored: Mon Aug 28 10:09:46 2017 -0700
Committer: Yufei Gu <[email protected]>
Committed: Mon Aug 28 10:11:01 2017 -0700

----------------------------------------------------------------------
 .../linux/resources/ResourceHandlerModule.java                 | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hadoop/blob/221c1bdf/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/resources/ResourceHandlerModule.java
----------------------------------------------------------------------
diff --git 
a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/resources/ResourceHandlerModule.java
 
b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/resources/ResourceHandlerModule.java
index a81a77b..3c61cd4 100644
--- 
a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/resources/ResourceHandlerModule.java
+++ 
b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/linux/resources/ResourceHandlerModule.java
@@ -268,11 +268,11 @@ public class ResourceHandlerModule {
       // Collect the valid subsystem names
       cgroupList.retainAll(validCGroups);
       if (!cgroupList.isEmpty()) {
-        if (candidate.isDirectory() && candidate.canWrite()) {
+        if (candidate.isDirectory()) {
           pathSubsystemMappings.put(candidate.getAbsolutePath(), cgroupList);
         } else {
-          LOG.warn("The following cgroup is not a directory or it is not"
-              + " writable" + candidate.getAbsolutePath());
+          LOG.warn("The following cgroup is not a directory " +
+              candidate.getAbsolutePath());
         }
       }
     }


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

Reply via email to