sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Szabolcs Vasas <vasas.szabo...@gmail.com>
Subject Re: Review Request 67873: Add Hive support to the new Parquet writing implementation
Date Mon, 16 Jul 2018 10:16:45 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67873/
-----------------------------------------------------------

(Updated July 16, 2018, 10:16 a.m.)


Review request for Sqoop.


Changes
-------

Unused import is removed.


Bugs: SQOOP-3335
    https://issues.apache.org/jira/browse/SQOOP-3335


Repository: sqoop-trunk


Description
-------

SQOOP-3328 adds a new Parquet reading and writing implementation to Sqoop it does not add
support to Hive Parquet imports. The task of this Jira is to add this missing functionality.


Diffs (updated)
-----

  src/java/org/apache/sqoop/hive/HiveTypes.java ad00535e5 
  src/java/org/apache/sqoop/hive/TableDefWriter.java 27d988c53 
  src/java/org/apache/sqoop/mapreduce/parquet/ParquetImportJobConfigurator.java eb6d08f8a

  src/java/org/apache/sqoop/mapreduce/parquet/hadoop/HadoopParquetImportJobConfigurator.java
3f35faf86 
  src/java/org/apache/sqoop/mapreduce/parquet/kite/KiteParquetImportJobConfigurator.java feb3bf19b

  src/java/org/apache/sqoop/tool/BaseSqoopTool.java e505c2671 
  src/java/org/apache/sqoop/tool/ImportTool.java 25c3f7031 
  src/test/org/apache/sqoop/TestParquetIncrementalImportMerge.java d8d3af40f 
  src/test/org/apache/sqoop/hive/TestHiveServer2ParquetImport.java PRE-CREATION 
  src/test/org/apache/sqoop/hive/TestHiveServer2TextImport.java 3d115ab3e 
  src/test/org/apache/sqoop/hive/TestHiveTypesForAvroTypeMapping.java PRE-CREATION 
  src/test/org/apache/sqoop/hive/TestTableDefWriter.java 3ea61f646 
  src/test/org/apache/sqoop/testutil/BaseSqoopTestCase.java ac6db0b14 
  src/test/org/apache/sqoop/tool/TestHiveServer2OptionValidations.java 4d3f93898 


Diff: https://reviews.apache.org/r/67873/diff/2/

Changes: https://reviews.apache.org/r/67873/diff/1-2/


Testing
-------

Executed unit and third party test cases.


Thanks,

Szabolcs Vasas


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message