----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69407/#review210756 -----------------------------------------------------------
Ship it! Hi Feró, Thank you for improving these tests, unit and third party tests were both successful. - Szabolcs Vasas On Nov. 21, 2018, 10:20 a.m., Fero Szabo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/69407/ > ----------------------------------------------------------- > > (Updated Nov. 21, 2018, 10:20 a.m.) > > > Review request for Sqoop, Boglarka Egyed and Szabolcs Vasas. > > > Bugs: SQOOP-3405 > https://issues.apache.org/jira/browse/SQOOP-3405 > > > Repository: sqoop-trunk > > > Description > ------- > > Breaking up the parameterized test classes into a per database basis. > Provides better readability, needed for proper test categorization (and thus, > for travis integration). > > > Diffs > ----- > > src/test/org/apache/sqoop/importjob/DatabaseAdapterFactory.java > PRE-CREATION > src/test/org/apache/sqoop/importjob/NumericTypesImportTest.java af310cbe > src/test/org/apache/sqoop/importjob/SplitByImportTest.java 90b7cbbd > > src/test/org/apache/sqoop/importjob/configuration/MSSQLServerImportJobTestConfiguration.java > 4ad7defe > > src/test/org/apache/sqoop/importjob/configuration/MySQLImportJobTestConfiguration.java > fbcbdebe > > src/test/org/apache/sqoop/importjob/numerictypes/MysqlNumericTypesImportTest.java > PRE-CREATION > > src/test/org/apache/sqoop/importjob/numerictypes/OracleNumericTypesImportTest.java > PRE-CREATION > > src/test/org/apache/sqoop/importjob/numerictypes/PostgresNumericTypesImportTest.java > PRE-CREATION > > src/test/org/apache/sqoop/importjob/numerictypes/SqlServerNumericTypesImportTest.java > PRE-CREATION > src/test/org/apache/sqoop/importjob/splitby/MysqlSplitByImportTest.java > PRE-CREATION > src/test/org/apache/sqoop/importjob/splitby/OracleSplitByImportTest.java > PRE-CREATION > src/test/org/apache/sqoop/importjob/splitby/PostgresSplitByImportTest.java > PRE-CREATION > src/test/org/apache/sqoop/importjob/splitby/SqlServerSplitByImportTest.java > PRE-CREATION > src/test/org/apache/sqoop/testutil/adapter/MSSQLServerDatabaseAdapter.java > 22567162 > src/test/org/apache/sqoop/testutil/adapter/MySqlDatabaseAdapter.java > ebd01468 > > > Diff: https://reviews.apache.org/r/69407/diff/5/ > > > Testing > ------- > > unit and 3rd party tests. > > > Thanks, > > Fero Szabo > >