-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68687/
-----------------------------------------------------------
(Updated Sept. 12, 2018, 2:53 p.m.)
Review request for Sqoop, Boglarka Egyed, daniel voros, and Szabolcs Vasas.
Bugs: SQOOP-3381
https://issues.apache.org/jira/browse/SQOOP-3381
Repository: sqoop-trunk
Description
-------
This change upgrades our parquet library to the newest version.
As we will need to register a data supplier in the fix for parquet decimal support (SQOOP-3382),
we will need a version that contains PARQUET-243. We need to upgrade the Parquet library to
a version that contains this fix and is compatible with Hadoop 3.0.
The only difference is found in the TestParquetImport class, the new implementation returns
a Utf8 object for Strings written out.
Diffs
-----
gradle.properties 0d30378d
gradle/sqoop-package.gradle 1a8d994d
ivy.xml 670cb32d
ivy/libraries.properties 8f3dab2b
src/java/org/apache/sqoop/avro/AvroUtil.java 1663b1d1
src/java/org/apache/sqoop/mapreduce/parquet/hadoop/HadoopParquetExportJobConfigurator.java
2180cc20
src/java/org/apache/sqoop/mapreduce/parquet/hadoop/HadoopParquetImportJobConfigurator.java
90b910a3
src/java/org/apache/sqoop/mapreduce/parquet/hadoop/HadoopParquetMergeJobConfigurator.java
66ebc5b8
src/test/org/apache/sqoop/TestParquetExport.java be1d8164
src/test/org/apache/sqoop/TestParquetImport.java 2810e318
src/test/org/apache/sqoop/TestParquetIncrementalImportMerge.java adad0cc1
src/test/org/apache/sqoop/hive/TestHiveServer2ParquetImport.java b55179a4
src/test/org/apache/sqoop/util/ParquetReader.java f1c2fe10
Diff: https://reviews.apache.org/r/68687/diff/3/
Testing (updated)
-------
Ant unit and 3rd party tests were successful.
gradlew test was successful
gradlew thirdPartyTest was successful as well. (it had 1 failed test at first run: HBaseImportTest#testAppendWithTimestampSucceeds)
Thanks,
Fero Szabo
|