This is an automated email from the ASF dual-hosted git repository. jhung pushed a commit to branch YARN-8200.branch3 in repository https://gitbox.apache.org/repos/asf/hadoop.git
commit 028a74bbaa41e55fa0a5986e33ca71432da0720a Author: Jian He <jia...@apache.org> AuthorDate: Wed Nov 1 17:00:32 2017 -0700 YARN-7396. NPE when accessing container logs due to null dirsHandler. Contributed by Jonathan Hung --- .../java/org/apache/hadoop/yarn/server/nodemanager/NodeManager.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/NodeManager.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/NodeManager.java index c7edec2..54a235c 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/NodeManager.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/NodeManager.java @@ -368,6 +368,8 @@ public class NodeManager extends CompositeService this.aclsManager = new ApplicationACLsManager(conf); + this.dirsHandler = new LocalDirsHandlerService(metrics); + boolean isDistSchedulingEnabled = conf.getBoolean(YarnConfiguration.DIST_SCHEDULING_ENABLED, YarnConfiguration.DEFAULT_DIST_SCHEDULING_ENABLED); @@ -391,7 +393,6 @@ public class NodeManager extends CompositeService // NodeManager level dispatcher this.dispatcher = createNMDispatcher(); - dirsHandler = new LocalDirsHandlerService(metrics); nodeHealthChecker = new NodeHealthCheckerService( getNodeHealthScriptRunner(conf), dirsHandler); --------------------------------------------------------------------- To unsubscribe, e-mail: common-commits-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-commits-h...@hadoop.apache.org