----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14022/#review26037 -----------------------------------------------------------
Ship it! Ship It! - Jarek Cecho On Sept. 10, 2013, 6:39 p.m., Raghav Gautam wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/14022/ > ----------------------------------------------------------- > > (Updated Sept. 10, 2013, 6:39 p.m.) > > > Review request for Sqoop. > > > 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 > ----- > > > 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 > >