----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58295/#review171576 -----------------------------------------------------------
Ship it! Ship It! - Anna Szonyi On April 10, 2017, 12:47 p.m., Boglarka Egyed wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58295/ > ----------------------------------------------------------- > > (Updated April 10, 2017, 12:47 p.m.) > > > Review request for Sqoop and Anna Szonyi. > > > Bugs: SQOOP-3169 > https://issues.apache.org/jira/browse/SQOOP-3169 > > > Repository: sqoop-trunk > > > Description > ------- > > Fixed and improved SQLServer Manual tests so that they can be executed at > once by setting DB connect string anf credentials via system properties. This > patch also contains some more improvement, like: > - ignored invalid test cases > - added database cleanup logic in tearDown part > - updated java docs > - removed unused imports > > > Diffs > ----- > > build.xml 73db28b272c50b4f76fef8421e6b9dfe5fed40f4 > src/test/com/cloudera/sqoop/hive/TestHiveImport.java > 33e0cc41f6f379bac2085431e0f1adc60bce6bce > src/test/com/cloudera/sqoop/manager/SQLServerManagerExportManualTest.java > 9a92479245fa35c210d8e49f847292ee53d6f9b1 > src/test/com/cloudera/sqoop/manager/SQLServerManagerImportManualTest.java > 1f69725da8408853ac55b1f316ce1b9ef015e674 > src/test/org/apache/sqoop/manager/sqlserver/MSSQLTestUtils.java > 851bf49614e829d07de252b83f4ad550d0cb043b > src/test/org/apache/sqoop/manager/sqlserver/ManagerCompatExport.java > 8c5176ad61aae61b96c7458d3b4b83dc11960268 > > src/test/org/apache/sqoop/manager/sqlserver/SQLServerDatatypeExportDelimitedFileManualTest.java > 099d7344beb428c58b32d926af5ea079211da490 > > src/test/org/apache/sqoop/manager/sqlserver/SQLServerDatatypeExportSequenceFileManualTest.java > 21676f02510693dcdd856a1d9dfba7d05eace023 > > src/test/org/apache/sqoop/manager/sqlserver/SQLServerDatatypeImportDelimitedFileManualTest.java > 519fb525bdbb167520368d404667036669925041 > > src/test/org/apache/sqoop/manager/sqlserver/SQLServerDatatypeImportSequenceFileManualTest.java > a0dad8a60b99d522ad3691e15b8b16c56e4b5858 > > src/test/org/apache/sqoop/manager/sqlserver/SQLServerHiveImportManualTest.java > 1999272181421a539318ed195ea4257f52b2ed08 > src/test/org/apache/sqoop/manager/sqlserver/SQLServerManagerManualTest.java > 1178e3c79de4d0b5c7a96c6ad7eb316ed15e47c4 > > src/test/org/apache/sqoop/manager/sqlserver/SQLServerMultiColsManualTest.java > 6a8ab51967237f471044b868615fdb3e057b1d92 > > src/test/org/apache/sqoop/manager/sqlserver/SQLServerMultiMapsManualTest.java > c9a5b5ef596cfc1b28948c2a071935dfb9500cde > > src/test/org/apache/sqoop/manager/sqlserver/SQLServerParseMethodsManualTest.java > cd05aecf1ae5bd79fb485325d58b33a73e9df290 > src/test/org/apache/sqoop/manager/sqlserver/SQLServerQueryManualTest.java > 0057ac9df562c8e92cf7b9014c5e4239886a8104 > src/test/org/apache/sqoop/manager/sqlserver/SQLServerSplitByManualTest.java > f85245ab8cdd66da983ac9017d356f251f22e7db > src/test/org/apache/sqoop/manager/sqlserver/SQLServerWhereManualTest.java > 10ae03b324b15f5ea0cc3cbbc04d3a5041233dd9 > > > Diff: https://reviews.apache.org/r/58295/diff/1/ > > > Testing > ------- > > ant clean test > > ant clean test -Dsqoop.thirdparty.lib.dir=3rdparty > -Dsqoop.test.sqlserver.connectstring.host_url=sqlserver_connectstring > -Dsqoop.test.sqlserver.database=databasename -Dms.sqlserver.username=username > -Dms.sqlserver.password=password -Dtestcase=SQLServer* > > > Thanks, > > Boglarka Egyed > >
