-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13397/
-----------------------------------------------------------
(Updated Aug. 16, 2013, 9:31 a.m.)
Review request for oozie.
Changes
-------
Addressed Rohini's review comments. testLauncher unit test WIP. Wasnt able to
figure out why loading sequence file is problematic sometimes
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
1514596
trunk/core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java
1514596
trunk/core/src/main/java/org/apache/oozie/action/hadoop/LauncherMapperHelper.java
1514596
trunk/core/src/main/java/org/apache/oozie/action/hadoop/PigActionExecutor.java
1514596
trunk/core/src/main/java/org/apache/oozie/action/hadoop/SqoopActionExecutor.java
1514596
trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestLauncher.java
1514596
trunk/sharelib/hive/src/main/java/org/apache/oozie/action/hadoop/HiveMain.java
1514596
trunk/sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/LauncherMapper.java
1514596
Diff: https://reviews.apache.org/r/13397/diff/
Testing
-------
Existing unit tests should pass, and E-2-E test pending.
Few tests failing with a common problem of Properties object (for
output/new-id/error etc) getting stored in the file with Timestamp. Hence on
retrieval, it gives NPE on the expected keys. WIP to fix that small part
Thanks,
Mona Chitnis
