-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11680/
-----------------------------------------------------------
Review request for oozie.
Description
-------
The patch is mostly moving some files from core to the Oozie sharelib. There
was some minor refactoring to fix some issues with tests. Otherwise, the only
actual code changes was to add the option to use the launcher jar or not
(oozie.action.use.launcher.jar).
This addresses bug OOZIE-1315.
https://issues.apache.org/jira/browse/OOZIE-1315
Diffs
-----
trunk/core/pom.xml 1490336
trunk/core/src/main/java/org/apache/oozie/action/hadoop/ActionStats.java
1490336
trunk/core/src/main/java/org/apache/oozie/action/hadoop/ActionType.java
1490336
trunk/core/src/main/java/org/apache/oozie/action/hadoop/FSLauncherURIHandler.java
1490336
trunk/core/src/main/java/org/apache/oozie/action/hadoop/HCatLauncherURIHandler.java
1490336
trunk/core/src/main/java/org/apache/oozie/action/hadoop/HiveActionExecutor.java
1490336
trunk/core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java
1490336
trunk/core/src/main/java/org/apache/oozie/action/hadoop/LauncherException.java
1490336
trunk/core/src/main/java/org/apache/oozie/action/hadoop/LauncherMain.java
1490336
trunk/core/src/main/java/org/apache/oozie/action/hadoop/LauncherMapper.java
1490336
trunk/core/src/main/java/org/apache/oozie/action/hadoop/LauncherMapperHelper.java
PRE-CREATION
trunk/core/src/main/java/org/apache/oozie/action/hadoop/LauncherURIHandler.java
1490336
trunk/core/src/main/java/org/apache/oozie/action/hadoop/LauncherURIHandlerFactory.java
1490336
trunk/core/src/main/java/org/apache/oozie/action/hadoop/MRStats.java 1490336
trunk/core/src/main/java/org/apache/oozie/action/hadoop/MapReduceActionExecutor.java
1490336
trunk/core/src/main/java/org/apache/oozie/action/hadoop/MapReduceMain.java
1490336
trunk/core/src/main/java/org/apache/oozie/action/hadoop/PigActionExecutor.java
1490336
trunk/core/src/main/java/org/apache/oozie/action/hadoop/PipesMain.java
1490336
trunk/core/src/main/java/org/apache/oozie/action/hadoop/PrepareActionsDriver.java
1490336
trunk/core/src/main/java/org/apache/oozie/action/hadoop/ShellMain.java
1490336
trunk/core/src/main/java/org/apache/oozie/action/hadoop/SqoopActionExecutor.java
1490336
trunk/core/src/main/java/org/apache/oozie/util/HCatURI.java 1490336
trunk/core/src/main/resources/oozie-default.xml 1490336
trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestFSPrepareActions.java
1490336
trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestHCatPrepareActions.java
1490336
trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestJavaActionExecutor.java
1490336
trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestLauncher.java
1490336
trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestMapReduceActionError.java
1490336
trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestMapReduceActionExecutor.java
1490336
trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestMapReduceActionExecutorUberJar.java
1490336
trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestPrepareActionsDriver.java
1490336
trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestShellActionExecutor.java
1490336
trunk/core/src/test/java/org/apache/oozie/command/wf/TestActionCheckXCommand.java
1490336
trunk/core/src/test/java/org/apache/oozie/command/wf/TestActionStartXCommand.java
1490336
trunk/core/src/test/java/org/apache/oozie/service/TestRecoveryService.java
1490336
trunk/core/src/test/java/org/apache/oozie/util/TestHCatURI.java 1490336
trunk/core/src/test/resources/hsqldb-oozie-site.xml 1490336
trunk/examples/pom.xml 1490336
trunk/examples/src/test/resources/hsqldb-oozie-site.xml 1490336
trunk/minitest/src/test/resources/hsqldb-oozie-site.xml 1490336
trunk/pom.xml 1490336
trunk/sharelib/distcp/pom.xml 1490336
trunk/sharelib/hcatalog/pom.xml 1490336
trunk/sharelib/hive/pom.xml 1490336
trunk/sharelib/hive/src/test/java/org/apache/oozie/action/hadoop/TestHiveActionExecutor.java
1490336
trunk/sharelib/oozie/pom.xml 1490336
trunk/sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/LauncherMapper.java
PRE-CREATION
trunk/sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/MapReduceMain.java
PRE-CREATION
trunk/sharelib/pig/pom.xml 1490336
trunk/sharelib/pig/src/test/java/org/apache/oozie/action/hadoop/TestPigActionExecutor.java
1490336
trunk/sharelib/sqoop/pom.xml 1490336
trunk/sharelib/sqoop/src/test/java/org/apache/oozie/action/hadoop/TestSqoopActionExecutor.java
1490336
trunk/sharelib/streaming/pom.xml 1490336
trunk/tools/src/test/resources/hsqldb-oozie-site.xml 1490336
trunk/webapp/pom.xml 1490336
Diff: https://reviews.apache.org/r/11680/diff/
Testing
-------
- Unit tests for oozie.action.use.launcher.jar
- I verified that it works correctly with oozie.action.use.launcher.jar set to
true and false.
Thanks,
Robert Kanter
