----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39927/#review105686 -----------------------------------------------------------
Ship it! Ship It! - Jarek Cecho On Nov. 9, 2015, 1:51 a.m., Colin Ma wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39927/ > ----------------------------------------------------------- > > (Updated Nov. 9, 2015, 1:51 a.m.) > > > Review request for Sqoop. > > > Repository: sqoop-sqoop2 > > > Description > ------- > > There are 2 problems should be fixed with MySql repository: > 1. Can't detect the repository version correctly. > 2. There should suspend several seconds when execute the job in the > integration test. > > > Diffs > ----- > > > common-test/src/main/java/org/apache/sqoop/common/test/db/MySQLProvider.java > 268e475 > > common-test/src/main/java/org/apache/sqoop/common/test/repository/MysqlRepositoryProvider.java > 229b339 > > repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositoryInsertUpdateDeleteSelectQuery.java > 3a4e80a > > repository/repository-mysql/src/main/java/org/apache/sqoop/repository/mysql/MySqlRepositoryHandler.java > fd3a3f2 > > Diff: https://reviews.apache.org/r/39927/diff/ > > > Testing > ------- > > > Thanks, > > Colin Ma > >
