----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68382/#review207747 -----------------------------------------------------------
Ship it! Hi Szabolcs, Thanks for taking care of this. The test PostgresMetaConnectIncrementalImportTest failed when I ran the third party test suite but succeeded when I ran it again alone. I guess it was not because of your patch though. Best, Nguyen - Nguyen Truong On Aug. 16, 2018, 2:37 p.m., Szabolcs Vasas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68382/ > ----------------------------------------------------------- > > (Updated Aug. 16, 2018, 2:37 p.m.) > > > Review request for Sqoop. > > > Bugs: SQOOP-3364 > https://issues.apache.org/jira/browse/SQOOP-3364 > > > Repository: sqoop-trunk > > > Description > ------- > > Apart from the Gradle version bump the change contains the following: > - html.destination type is modified to file to avoid deprecation warning > - the task wrapper is replaced with wrapper {} to avoid deprecation warning > - enableFeaturePreview('STABLE_PUBLISHING') is added to settings.gradle to > avoid deprecation warning. This is a change I could not test since we cannot > publish to maven repo now. However in a case of a future release we should > test it as described here: > https://docs.gradle.org/4.9/userguide/publishing_maven.html#publishing_maven:deferred_configuration > - The HBase test cases failed at first because the regionserver web ui was > not able to start up most probably because of a bad version of a Jetty class > on the classpath. However we do not need the regionserver web ui for the > Sqoop tests so instead of playing around with libraries I disabled it just > like we have already disabled the master web ui. > > > Diffs > ----- > > build.gradle 709172cc0 > gradle/wrapper/gradle-wrapper.jar 99340b4ad18d3c7e764794d300ffd35017036793 > gradle/wrapper/gradle-wrapper.properties 90a06cec7 > settings.gradle 7d64af500 > src/test/org/apache/sqoop/hbase/HBaseTestCase.java 87fce34a8 > > > Diff: https://reviews.apache.org/r/68382/diff/1/ > > > Testing > ------- > > Executed unit and third party test suite successfully. > > > Thanks, > > Szabolcs Vasas > >
