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

(Updated July 26, 2012, 10:41 a.m.)


Review request for Sqoop.


Changes
-------

1. movee all classes except testcase into org.apache.sqoop from 
com.cloudera.sqoop.
2. updated license header from Cloudera to Apache Software Foundation.
3. moved mapreduce job configuration setting to PGBulkloadExportJob from 
PGBulkloadManager.
4. fixed PGBulkloadManagerManualTest to pass tests against current trunk.
5. trivial fixes to pass checkstyle.


Description
-------

Patch for SQOOP-390
https://issues.apache.org/jira/browse/SQOOP-390


This addresses bug SQOOP-390.
    https://issues.apache.org/jira/browse/SQOOP-390


Diffs (updated)
-----

  /src/java/org/apache/sqoop/manager/PGBulkloadManager.java PRE-CREATION 
  /src/java/org/apache/sqoop/mapreduce/AutoProgressReducer.java PRE-CREATION 
  /src/java/org/apache/sqoop/mapreduce/PGBulkloadExportJob.java PRE-CREATION 
  /src/java/org/apache/sqoop/mapreduce/PGBulkloadExportMapper.java PRE-CREATION 
  /src/java/org/apache/sqoop/mapreduce/PGBulkloadExportReducer.java 
PRE-CREATION 
  /src/test/com/cloudera/sqoop/manager/PGBulkloadManagerManualTest.java 
PRE-CREATION 

Diff: https://reviews.apache.org/r/2724/diff/


Testing
-------

This patch include the test class PGBulkloadManagerTest.
I've tested "ant test" and passed.


Thanks,

Masatake Iwasaki

Reply via email to