sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jarek Cecho" <jar...@apache.org>
Subject Review Request: SQOOP-601: Support custom schemas in PostgreSQL Connector
Date Wed, 12 Sep 2012 12:09:05 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for Sqoop.


This patch is adding support for custom schemas into PostgreSQL manager. Changes for import
job were very simple and rather straightforward. On the other side, changes for export job
were little tricky, but I believe that I've succeeded without breaking backward compatibility.


  src/docs/user/connectors.txt a93f14e 
  src/java/org/apache/sqoop/manager/CatalogQueryManager.java 5f2f89f 
  src/java/org/apache/sqoop/manager/PostgresqlManager.java d18321c 
  src/java/org/apache/sqoop/manager/SqlManager.java ea961cd 
  src/java/org/apache/sqoop/mapreduce/JdbcExportJob.java b574f82 
  src/java/org/apache/sqoop/mapreduce/JdbcUpdateExportJob.java 7be5ed9 
  src/java/org/apache/sqoop/mapreduce/JdbcUpsertExportJob.java f299f98 
  src/java/org/apache/sqoop/mapreduce/db/DataDrivenDBRecordReader.java 38e9fb9 
  src/test/com/cloudera/sqoop/ThirdPartyTests.java eeab7f3 
  src/test/com/cloudera/sqoop/manager/PostgresqlExportTest.java PRE-CREATION 
  src/test/com/cloudera/sqoop/manager/PostgresqlImportTest.java PRE-CREATION 
  src/test/com/cloudera/sqoop/manager/PostgresqlTest.java 0dfd1fc 

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


I've added a lot of third party tests (e.g. they won't be run automatically during "ant test")
to properly test various combinations of parameters. You can invoke those tests issuing one
of following commands:

ant test -Dtestcase=PostgresqlImportTest
ant test -Dtestcase=PostgresqlExportTest

I've tested compatibility by running ant test -Dhadoopversion={20,100,23,200}


Jarek Cecho

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