You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@oozie.apache.org by pb...@apache.org on 2017/03/22 11:23:18 UTC
[13/50] [abbrv] oozie git commit: Remote comments from a testcase in
TestJavaActionExecutor
Remote comments from a testcase in TestJavaActionExecutor
Change-Id: Idc32bf4de5a55b94f47d7b932c6b98fedb885784
Project: http://git-wip-us.apache.org/repos/asf/oozie/repo
Commit: http://git-wip-us.apache.org/repos/asf/oozie/commit/85e65486
Tree: http://git-wip-us.apache.org/repos/asf/oozie/tree/85e65486
Diff: http://git-wip-us.apache.org/repos/asf/oozie/diff/85e65486
Branch: refs/heads/oya
Commit: 85e654869220f5f55fc9d51f97a296f7c4d9690f
Parents: 3922f50
Author: Peter Bacsko <pb...@cloudera.com>
Authored: Mon Jan 23 12:12:58 2017 +0100
Committer: Peter Bacsko <pb...@cloudera.com>
Committed: Mon Jan 23 12:12:58 2017 +0100
----------------------------------------------------------------------
.../org/apache/oozie/action/hadoop/TestJavaActionExecutor.java | 4 ----
1 file changed, 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/oozie/blob/85e65486/core/src/test/java/org/apache/oozie/action/hadoop/TestJavaActionExecutor.java
----------------------------------------------------------------------
diff --git a/core/src/test/java/org/apache/oozie/action/hadoop/TestJavaActionExecutor.java b/core/src/test/java/org/apache/oozie/action/hadoop/TestJavaActionExecutor.java
index bd53978..89f3099 100644
--- a/core/src/test/java/org/apache/oozie/action/hadoop/TestJavaActionExecutor.java
+++ b/core/src/test/java/org/apache/oozie/action/hadoop/TestJavaActionExecutor.java
@@ -257,7 +257,6 @@ public class TestJavaActionExecutor extends ActionExecutorTestCase {
Configuration actionConf = ae.createBaseHadoopConf(context, actionXml);
ae.setupActionConf(actionConf, context, actionXml, getFsTestCaseDir());
-
conf = ae.createLauncherConf(getFileSystem(), context, action, actionXml, actionConf);
ae.setupLauncherConf(conf, actionXml, getFsTestCaseDir(), context);
assertEquals("MAIN-CLASS", actionConf.get("oozie.action.java.main", "null"));
@@ -266,9 +265,6 @@ public class TestJavaActionExecutor extends ActionExecutorTestCase {
assertTrue(conf.get("mapreduce.map.java.opts").contains("JAVA-OPTS"));
assertEquals(Arrays.asList("A1", "A2"), Arrays.asList(LauncherMapper.getMainArguments(conf)));
- // FIXME - this file exists - must use the correct path
- // assertTrue(getFileSystem().exists(new Path(context.getActionDir(), LauncherMapper.ACTION_CONF_XML)));
-
actionXml = XmlUtils.parseXml("<java>" + "<job-tracker>" + getJobTrackerUri() + "</job-tracker>" +
"<name-node>" + getNameNodeUri() + "</name-node> <configuration>" +
"<property><name>mapred.job.queue.name</name><value>AQ</value></property>" +