-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13397/
-----------------------------------------------------------

(Updated Aug. 19, 2013, 7:40 a.m.)


Review request for oozie.


Changes
-------

unit tests Pig/Hive/Sqoop involving externalIds and outputdata are failing. 
Reasons
1. data in mem obj might be pertaining to launcher job and not of the actual 
action.
2. Reading back sequencefile Text value into String, not delimiting correctly 
e.g. in error-properties, error.message and error.code are two properties but 
is getting retrieved as one single string.


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

Reply via email to