sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Sandish Kumar HN <sanysand...@gmail.com>
Subject Re: Review Request 61372: Expanded Metastore support for MySql, Oracle, Postgresql, MSSql, and DB2
Date Fri, 15 Sep 2017 10:12:25 GMT


> On Aug. 22, 2017, 2:49 p.m., Sandish Kumar HN wrote:
> > Is it possible to add few more test cases??
> 
> Zach Berkowitz wrote:
>     Hi Sandish, Do you have any specific aspects you'd like to see tested more?
> 
> Boglarka Egyed wrote:
>     Hi Sandish,
>     
>     Could you please provide some more details regarding the missing test cases? Or please
let me know if you are fine with the updated patch as it is.
>     
>     I would like to proceed with committing this change.
>     
>     Thanks,
>     Bogi

I think there are enough testcases for this patch. I'm ok with current test case's


- Sandish Kumar


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


On Aug. 25, 2017, 11:53 a.m., Zach Berkowitz wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/61372/
> -----------------------------------------------------------
> 
> (Updated Aug. 25, 2017, 11:53 a.m.)
> 
> 
> Review request for Sqoop, Boglarka Egyed, Jarek Cecho, and Anna Szonyi.
> 
> 
> Bugs: SQOOP-3216
>     https://issues.apache.org/jira/browse/SQOOP-3216
> 
> 
> Repository: sqoop-trunk
> 
> 
> Description
> -------
> 
> Expanded Metastore support for MySql, Oracle, Postgresql, MSSql, and DB2
> 
> 
> Diffs
> -----
> 
>   src/docs/user/metastore-purpose.txt e7eb23d3 
>   src/docs/user/saved-jobs.txt e8757801 
>   src/java/com/cloudera/sqoop/metastore/hsqldb/AutoHsqldbStorage.java 259d9f63 
>   src/java/com/cloudera/sqoop/metastore/hsqldb/HsqldbJobStorage.java 083e2a37 
>   src/java/org/apache/sqoop/SqoopOptions.java 2eb3d8a4 
>   src/java/org/apache/sqoop/manager/CubridManager.java 5a1a0e83 
>   src/java/org/apache/sqoop/manager/Db2Manager.java 61b6868d 
>   src/java/org/apache/sqoop/manager/HsqldbManager.java 9b9c5822 
>   src/java/org/apache/sqoop/manager/JdbcDrivers.java PRE-CREATION 
>   src/java/org/apache/sqoop/manager/MySQLManager.java 3c2276fe 
>   src/java/org/apache/sqoop/manager/NetezzaManager.java 0ac77175 
>   src/java/org/apache/sqoop/manager/OracleManager.java 2f4585cc 
>   src/java/org/apache/sqoop/manager/PostgresqlManager.java 44e041ad 
>   src/java/org/apache/sqoop/manager/SQLServerManager.java 9a3d9183 
>   src/java/org/apache/sqoop/manager/SupportedManagers.java 8a6037af 
>   src/java/org/apache/sqoop/metastore/JobStorageFactory.java 2edc33b8 
>   src/java/org/apache/sqoop/metastore/hsqldb/AutoHsqldbStorage.java 49e30319 
>   src/java/org/apache/sqoop/metastore/hsqldb/HsqldbJobStorage.java a0f29fd0 
>   src/java/org/apache/sqoop/tool/BaseSqoopTool.java 1564bdcb 
>   src/java/org/apache/sqoop/tool/JobTool.java 054e274f 
>   src/test/com/cloudera/sqoop/TestIncrementalImport.java 52a55b78 
>   src/test/com/cloudera/sqoop/metastore/JobToolTestBase.java PRE-CREATION 
>   src/test/com/cloudera/sqoop/metastore/MetaConnectIncrementalImportTestBase.java PRE-CREATION

>   src/test/com/cloudera/sqoop/metastore/SavedJobsTestBase.java PRE-CREATION 
>   src/test/com/cloudera/sqoop/metastore/TestSavedJobs.java 61d8c97d 
>   src/test/com/cloudera/sqoop/metastore/db2/DB2JobToolTest.java PRE-CREATION 
>   src/test/com/cloudera/sqoop/metastore/db2/DB2MetaConnectIncrementalImportTest.java
PRE-CREATION 
>   src/test/com/cloudera/sqoop/metastore/db2/DB2SavedJobsTest.java PRE-CREATION 
>   src/test/com/cloudera/sqoop/metastore/hsqldb/HsqldbMetaConnectIncrementalImportTest.java
PRE-CREATION 
>   src/test/com/cloudera/sqoop/metastore/hsqldb/HsqldbSavedJobsTest.java PRE-CREATION

>   src/test/com/cloudera/sqoop/metastore/mysql/MySqlJobToolTest.java PRE-CREATION 
>   src/test/com/cloudera/sqoop/metastore/mysql/MySqlMetaConnectIncrementalImportTest.java
PRE-CREATION 
>   src/test/com/cloudera/sqoop/metastore/mysql/MySqlSavedJobsTest.java PRE-CREATION 
>   src/test/com/cloudera/sqoop/metastore/oracle/OracleJobToolTest.java PRE-CREATION 
>   src/test/com/cloudera/sqoop/metastore/oracle/OracleMetaConnectIncrementalImportTest.java
PRE-CREATION 
>   src/test/com/cloudera/sqoop/metastore/oracle/OracleSavedJobsTest.java PRE-CREATION

>   src/test/com/cloudera/sqoop/metastore/postgres/PostgresJobToolTest.java PRE-CREATION

>   src/test/com/cloudera/sqoop/metastore/postgres/PostgresMetaConnectIncrementalImportTest.java
PRE-CREATION 
>   src/test/com/cloudera/sqoop/metastore/postgres/PostgresSavedJobsTest.java PRE-CREATION

>   src/test/com/cloudera/sqoop/metastore/sqlserver/SqlServerJobToolTest.java PRE-CREATION

>   src/test/com/cloudera/sqoop/metastore/sqlserver/SqlServerMetaConnectIncrementalImportTest.java
PRE-CREATION 
>   src/test/com/cloudera/sqoop/metastore/sqlserver/SqlServerSavedJobsTest.java PRE-CREATION

>   src/test/findbugsExcludeFile.xml a27ec378 
> 
> 
> Diff: https://reviews.apache.org/r/61372/diff/5/
> 
> 
> Testing
> -------
> 
> Three test classes SavedJobsTest, MetaConnectIncrementalImportTest, and JobToolTest all
pass for all supported databases.
> 
> 
> Thanks,
> 
> Zach Berkowitz
> 
>


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