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

Review request for hive.


Repository: hive-git


Description
-------

HIVE-8119: Implement Date in ParquetSerde

This patch map the Date in Hive to INT32 in Parquet, based on the Parquet 
Logical Type Definitions in 
https://github.com/apache/incubator-parquet-format/blob/master/LogicalTypes.md


Diffs
-----

  data/files/parquet_types.txt 31a10c9 
  ql/src/java/org/apache/hadoop/hive/ql/io/parquet/convert/ETypeConverter.java 
377e362 
  
ql/src/java/org/apache/hadoop/hive/ql/io/parquet/convert/HiveSchemaConverter.java
 e5bd70c 
  
ql/src/java/org/apache/hadoop/hive/ql/io/parquet/serde/ArrayWritableObjectInspector.java
 bb066af 
  ql/src/java/org/apache/hadoop/hive/ql/io/parquet/serde/ParquetHiveSerDe.java 
9199127 
  
ql/src/java/org/apache/hadoop/hive/ql/io/parquet/write/DataWritableWriter.java 
1d83bf3 
  ql/src/test/org/apache/hadoop/hive/ql/io/parquet/TestHiveSchemaConverter.java 
f232c57 
  ql/src/test/queries/clientnegative/parquet_date.q 89d3602 
  ql/src/test/queries/clientpositive/parquet_types.q 806db24 
  ql/src/test/results/clientnegative/parquet_date.q.out d1c38d6 
  ql/src/test/results/clientpositive/parquet_types.q.out dc5ceb0 

Diff: https://reviews.apache.org/r/30717/diff/


Testing
-------

UT passed. 2 tests are added


Thanks,

Dong Chen

Reply via email to