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 <[email protected]> Authored: Mon Jan 23 12:12:58 2017 +0100 Committer: Peter Bacsko <[email protected]> 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>" +
