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

Review request for Sqoop.


Description
-------

I've created abstract database handler that can be used in tests to interact 
with remote databases (create tables, insert data, ...). I've also provided 
concrete implementation based on Derby. I'll add more handlers later if this 
patch gets accepted.


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


Diffs
-----

  pom.xml ee013f2c63ff85e4f3b89d75ad860165b0fd0e67 
  test/pom.xml 66382b6fd7d17d4b345f5960082fb39e2ee100d4 
  test/src/main/java/org/apache/sqoop/test/db/DatabaseProvider.java 
PRE-CREATION 
  test/src/main/java/org/apache/sqoop/test/db/DerbyProvider.java PRE-CREATION 
  
test/src/main/java/org/apache/sqoop/test/minicluster/TomcatSqoopMiniCluster.java
 5fa294a2adc4a2f6855fda6e8853009b53554710 
  test/src/test/java/org/apache/sqoop/integration/TomcatTestCase.java 
eacf3045580cf93879905b706f1114a407283bad 
  
test/src/test/java/org/apache/sqoop/integration/connector/ConnectorTestCase.java
 PRE-CREATION 
  
test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/TableImportTest.java
 PRE-CREATION 
  test/src/test/resources/log4j.properties PRE-CREATION 

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


Testing
-------

I've provided first "real" integration test that this utilizing newly created 
database handler to set up the environment and move some data. Sadly we 
currently do not have convenient client API that we can use, so the 
implementation for connection and job creation is quite "hacky". I'll improve 
that once we will have reasonable client API.


Thanks,

Jarek Cecho

Reply via email to