sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Masatake Iwasaki" <iwasak...@nttdata.co.jp>
Subject Re: Review Request: SQOOP-390: PostgreSQL connector for direct export with pg_bulkload
Date Thu, 26 Jul 2012 10:41:07 GMT

-----------------------------------------------------------
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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message