----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67429/#review204846 -----------------------------------------------------------
Ship it! Thanks for updating the patch! Tests passed. - Boglarka Egyed On June 15, 2018, 1:03 p.m., Chris Teoh wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67429/ > ----------------------------------------------------------- > > (Updated June 15, 2018, 1:03 p.m.) > > > Review request for Sqoop. > > > Repository: sqoop-trunk > > > Description > ------- > > Added simple test for GDG dataset. Fixed bug triggered by recent Parquet > patch. > > > Diffs > ----- > > build.xml a85705fa > src/java/org/apache/sqoop/manager/MainframeManager.java 4e8be155 > > src/scripts/thirdpartytest/docker-compose/sqoop-thirdpartytest-db-services.yml > 2f4a07f6 > src/test/org/apache/sqoop/manager/mainframe/MainframeManagerImportTest.java > PRE-CREATION > src/test/org/apache/sqoop/manager/mainframe/MainframeTestUtil.java > PRE-CREATION > > > Diff: https://reviews.apache.org/r/67429/diff/8/ > > > Testing > ------- > > Running the integration and unit test on a patched version of trunk. > > > Thanks, > > Chris Teoh > >
