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

(Updated Feb. 27, 2013, 11:48 p.m.)


Review request for Sqoop and Jarek Cecho.


Changes
-------

This takes care of the latest comments from Jarcec.   Checkstyle violations 
have been fixed.   Validated the runs of unit tests.    
Connector Document has been updated and uploaded to the JIRA.   That explains 
the update table restriction.

Thanks Jarcec for a thorough review. 


Description
-------

This addresses SQOOP-846 (provide a Netezza connector)


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


Diffs (updated)
-----

  src/docs/user/connectors.txt 7dd2a2e 
  src/java/org/apache/sqoop/lib/DelimiterSet.java 4e9bcab 
  src/java/org/apache/sqoop/manager/DefaultManagerFactory.java 54eb258 
  src/java/org/apache/sqoop/manager/DirectNetezzaManager.java PRE-CREATION 
  src/java/org/apache/sqoop/manager/MySQLUtils.java ef18818 
  src/java/org/apache/sqoop/manager/NetezzaManager.java PRE-CREATION 
  
src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaDBDataSliceSplitter.java 
PRE-CREATION 
  
src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaExternalTableExportMapper.java
 PRE-CREATION 
  
src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaExternalTableImportMapper.java
 PRE-CREATION 
  
src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaExternalTableRecordExportMapper.java
 PRE-CREATION 
  
src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaExternalTableTextExportMapper.java
 PRE-CREATION 
  
src/java/org/apache/sqoop/mapreduce/db/netezza/NetezzaJDBCStatementRunner.java 
PRE-CREATION 
  src/test/com/cloudera/sqoop/manager/DirectMySQLExportTest.java eac7836 
  src/test/com/cloudera/sqoop/manager/DirectNetezzaExportManualTest.java 
PRE-CREATION 
  src/test/com/cloudera/sqoop/manager/NetezzaImportManualTest.java PRE-CREATION 
  src/test/com/cloudera/sqoop/manager/NetezzaTestUtils.java PRE-CREATION 

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


Testing
-------

Ran all sqoop tests.   Ran Netezza manual tests against Netezza VMs version 6 
and 7


Thanks,

Venkat Ranganathan

Reply via email to