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

(Updated June 24, 2013, 11:29 p.m.)


Review request for Sqoop.


Changes
-------

Manager logic is moved to DirectPostgresqlManager.
Tests added.
Trivial fixes based on checkstyle.


Description
-------

Export with PostgreSQL Copy API can be realized in a straitforward way. 
Attached file is not complete patch but sample implementation without tests and 
consideration for corner cases.
Though this code resolves dependency for PostgreSQL JDBC by ivy, is it 
acceptable for the project?


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


Diffs (updated)
-----

  ivy.xml 750adfc 
  ivy/libraries.properties 430d554 
  src/java/org/apache/sqoop/manager/DirectPostgresqlManager.java c085218 
  src/java/org/apache/sqoop/mapreduce/PostgreSQLCopyExportJob.java PRE-CREATION 
  src/java/org/apache/sqoop/mapreduce/PostgreSQLCopyMapper.java PRE-CREATION 
  src/test/com/cloudera/sqoop/manager/DirectPostgreSQLExportManualTest.java 
PRE-CREATION 

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


Testing
-------


Thanks,

Masatake Iwasaki

Reply via email to