Skip to content

Commit f6fea5d

Browse files
authored
MAPREDUCE-7468. [Addendum] Fix TestMapReduceChildJVM unit tests. (#6451)
1 parent 6ebce65 commit f6fea5d

File tree

1 file changed

+3
-2
lines changed
  • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/test/java/org/apache/hadoop/mapreduce/v2/app/job/impl

1 file changed

+3
-2
lines changed

hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/test/java/org/apache/hadoop/mapreduce/v2/app/job/impl/TestMapReduceChildJVM.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -108,6 +108,7 @@ private void testReduceCommandLine(Configuration conf)
108108

109109
MyMRApp app = new MyMRApp(0, 1, true, this.getClass().getName(), true);
110110
conf.setBoolean(MRConfig.MAPREDUCE_APP_SUBMISSION_CROSS_PLATFORM, true);
111+
conf.setBoolean(MRJobConfig.MAPREDUCE_JVM_ADD_OPENS_JAVA_OPT, true);
111112
Job job = app.submit(conf);
112113
app.waitForState(job, JobState.SUCCEEDED);
113114
app.verifyCompleted();
@@ -149,7 +150,7 @@ private void testReduceCommandLine(Configuration conf)
149150
}
150151

151152
@Test (timeout = 30000)
152-
public void testCommandLineWithLog4JConifg() throws Exception {
153+
public void testCommandLineWithLog4JConfig() throws Exception {
153154

154155
MyMRApp app = new MyMRApp(1, 0, true, this.getClass().getName(), true);
155156
Configuration conf = new Configuration();
@@ -165,7 +166,7 @@ public void testCommandLineWithLog4JConifg() throws Exception {
165166
"[" + MRApps.crossPlatformify("JAVA_HOME") + "/bin/java" +
166167
" -Djava.net.preferIPv4Stack=true" +
167168
" -Dhadoop.metrics.log.level=WARN " +
168-
" -Xmx820m <ADD_OPENS> -Djava.io.tmpdir=" + MRApps.crossPlatformify("PWD") + "/tmp" +
169+
" -Xmx820m -Djava.io.tmpdir=" + MRApps.crossPlatformify("PWD") + "/tmp" +
169170
" -Dlog4j.configuration=" + testLogPropertieFile +
170171
" -Dyarn.app.container.log.dir=<LOG_DIR>" +
171172
" -Dyarn.app.container.log.filesize=0" +

0 commit comments

Comments
 (0)