sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Raghav Gautam" <raghavgau...@gmail.com>
Subject Re: Review Request 14022: SQOOP-974 Support for stage table
Date Tue, 10 Sep 2013 18:39:25 GMT

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

(Updated Sept. 10, 2013, 11:39 a.m.)


Review request for Sqoop.


Changes
-------

Updated the patch according to the review comments.


Bugs: sqoop-974
    https://issues.apache.org/jira/browse/sqoop-974


Repository: sqoop-sqoop2


Description
-------

This patch adds support for stage table to Sqoop2.


Diffs (updated)
-----

  connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/GenericJdbcConnectorError.java
671bb4a 
  connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/GenericJdbcExecutor.java
75cf9d9 
  connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/GenericJdbcExportDestroyer.java
588e236 
  connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/GenericJdbcExportInitializer.java
7212843 
  connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/GenericJdbcValidator.java
4e24517 
  connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/configuration/ExportTableForm.java
a311c06 
  connector/connector-generic-jdbc/src/main/resources/generic-jdbc-connector-resources.properties
0950e32 
  connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/GenericJdbcExecutorTest.java
PRE-CREATION 
  connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestExportInitializer.java
f83aaa2 
  test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/exports/TableStagedExportTest.java
PRE-CREATION 

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


Testing
-------

Unit tests as well as integration tests have been added.
All the tests pass.
Feature was manually tested.


Thanks,

Raghav Gautam


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