Repository: hadoop
Updated Branches:
  refs/heads/trunk 08ff1586d -> 49c747ab1


YARN-8039. Clean up log dir configuration in 
TestLinuxContainerExecutorWithMocks.testStartLocalizer. Contributed by Miklos 
Szegedi.


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

Branch: refs/heads/trunk
Commit: 49c747ab187d0650143205ba57ca19607ec4c6bd
Parents: 08ff158
Author: Yufei Gu <[email protected]>
Authored: Fri Mar 16 16:25:33 2018 -0700
Committer: Yufei Gu <[email protected]>
Committed: Fri Mar 16 16:25:33 2018 -0700

----------------------------------------------------------------------
 .../server/nodemanager/TestLinuxContainerExecutorWithMocks.java  | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hadoop/blob/49c747ab/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/TestLinuxContainerExecutorWithMocks.java
----------------------------------------------------------------------
diff --git 
a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/TestLinuxContainerExecutorWithMocks.java
 
b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/TestLinuxContainerExecutorWithMocks.java
index e7a333e..a110f10 100644
--- 
a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/TestLinuxContainerExecutorWithMocks.java
+++ 
b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/TestLinuxContainerExecutorWithMocks.java
@@ -150,6 +150,7 @@ public class TestLinuxContainerExecutorWithMocks {
     LinuxContainerRuntime linuxContainerRuntime;
     LinuxContainerRuntime mockLinuxContainerRuntime;
 
+    conf.set(YarnConfiguration.NM_LOG_DIRS, "src/test/resources");
     setupMockExecutor(MOCK_EXECUTOR, conf);
     linuxContainerRuntime = new DefaultLinuxContainerRuntime(
         PrivilegedOperationExecutor.getInstance(conf));
@@ -281,7 +282,8 @@ public class TestLinuxContainerExecutorWithMocks {
       Assert.assertEquals(result.get(13),
           "-Dlog4j.configuration=container-log4j.properties" );
       Assert.assertEquals(result.get(14),
-          
"-Dyarn.app.container.log.dir=${yarn.log.dir}/userlogs/application_0/12345");
+          String.format("-Dyarn.app.container.log.dir=%s/application_0/12345",
+          mockExec.getConf().get(YarnConfiguration.NM_LOG_DIRS)));
       Assert.assertEquals(result.get(15),
           "-Dyarn.app.container.log.filesize=0");
       Assert.assertEquals(result.get(16), "-Dhadoop.root.logger=INFO,CLA");


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

Reply via email to