----------------------------------------------------------- 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