-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13397/
-----------------------------------------------------------
(Updated Aug. 22, 2013, 11:49 p.m.)
Review request for oozie.
Changes
-------
Addressed comments.
FYI - HiveMain and SqoopMain have been changed to store the child Hadoop Job
IDs under context.externalChildIDs. Earlier they were being stored redundantly
as context.externalChildIDs as well as context.action.data (output.properties).
Hence some change reflected in Main classes as well as the unit tests
Bugs: OOZIE-1486
https://issues.apache.org/jira/browse/OOZIE-1486
Repository: oozie
Description
-------
See JIRA Description
Diffs (updated)
-----
trunk/core/src/main/java/org/apache/oozie/action/hadoop/HiveActionExecutor.java
1516594
trunk/core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java
1516594
trunk/core/src/main/java/org/apache/oozie/action/hadoop/LauncherMapperHelper.java
1516594
trunk/core/src/main/java/org/apache/oozie/action/hadoop/PigActionExecutor.java
1516594
trunk/core/src/main/java/org/apache/oozie/action/hadoop/SqoopActionExecutor.java
1516594
trunk/core/src/test/java/org/apache/oozie/action/hadoop/LauncherMainTester.java
1516594
trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestLauncher.java
1516594
trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestMapReduceMain.java
1516594
trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestPipesMain.java
1516594
trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestShellActionExecutor.java
1516594
trunk/core/src/test/java/org/apache/oozie/command/wf/TestActionCheckXCommand.java
1516594
trunk/core/src/test/java/org/apache/oozie/command/wf/TestActionStartXCommand.java
1516594
trunk/core/src/test/java/org/apache/oozie/service/TestRecoveryService.java
1516594
trunk/sharelib/hive/src/main/java/org/apache/oozie/action/hadoop/HiveMain.java
1516594
trunk/sharelib/hive/src/test/java/org/apache/oozie/action/hadoop/TestHiveActionExecutor.java
1516594
trunk/sharelib/hive/src/test/java/org/apache/oozie/action/hadoop/TestHiveMain.java
1516594
trunk/sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/LauncherMapper.java
1516594
trunk/sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/MapReduceMain.java
1516594
trunk/sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/ShellMain.java
1516594
trunk/sharelib/pig/src/main/java/org/apache/oozie/action/hadoop/PigMain.java
1516594
trunk/sharelib/pig/src/main/java/org/apache/oozie/action/hadoop/PigMainWithOldAPI.java
1516594
trunk/sharelib/pig/src/test/java/org/apache/oozie/action/hadoop/TestPigActionExecutor.java
1516594
trunk/sharelib/pig/src/test/java/org/apache/oozie/action/hadoop/TestPigMain.java
1516594
trunk/sharelib/sqoop/src/main/java/org/apache/oozie/action/hadoop/SqoopMain.java
1516594
trunk/sharelib/sqoop/src/test/java/org/apache/oozie/action/hadoop/TestSqoopActionExecutor.java
1516594
trunk/sharelib/streaming/src/test/java/org/apache/oozie/action/hadoop/TestMapReduceActionExecutor.java
1516594
trunk/sharelib/streaming/src/test/java/org/apache/oozie/action/hadoop/TestStreamingMain.java
1516594
Diff: https://reviews.apache.org/r/13397/diff/
Testing
-------
E-2-E test with Pig stats verified.
Thanks,
Mona Chitnis
