You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Karen Coppage via Review Board <no...@reviews.apache.org> on 2020/02/06 14:12:03 UTC
Re: Review Request 72084: HIVE-21216: Write Parquet INT64 timestamp
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/72084/
-----------------------------------------------------------
(Updated Feb. 6, 2020, 2:12 p.m.)
Review request for hive and Marta Kuczora.
Bugs: HIVE-21216
https://issues.apache.org/jira/browse/HIVE-21216
Repository: hive-git
Description
-------
This patch enables Hive to start writing int64 timestamps in Parquet.
Diffs (updated)
-----
common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 12a022c590
ql/src/java/org/apache/hadoop/hive/ql/io/parquet/MapredParquetOutputFormat.java ba235f761d
ql/src/java/org/apache/hadoop/hive/ql/io/parquet/convert/HiveSchemaConverter.java 21bfb2e1a2
ql/src/java/org/apache/hadoop/hive/ql/io/parquet/timestamp/ParquetTimestampUtils.java 9ce07e74f3
ql/src/java/org/apache/hadoop/hive/ql/io/parquet/write/DataWritableWriteSupport.java 8acde81a3d
ql/src/java/org/apache/hadoop/hive/ql/io/parquet/write/DataWritableWriter.java bd519eb66e
ql/src/test/org/apache/hadoop/hive/ql/io/parquet/HiveParquetSchemaTestUtils.java 181894f106
ql/src/test/org/apache/hadoop/hive/ql/io/parquet/TestDataWritableWriter.java b242392a9a
ql/src/test/org/apache/hadoop/hive/ql/io/parquet/TestHiveSchemaConverter.java dc80af1b76
ql/src/test/org/apache/hadoop/hive/ql/io/parquet/serde/TestParquetTimestampUtils.java 9ea78508ee
ql/src/test/queries/clientpositive/parquet_int64_timestamp.q PRE-CREATION
ql/src/test/queries/clientpositive/parquet_int64_timestamp_int96_compatibility.q PRE-CREATION
ql/src/test/results/clientpositive/parquet_int64_timestamp.q.out PRE-CREATION
ql/src/test/results/clientpositive/parquet_int64_timestamp_int96_compatibility.q.out PRE-CREATION
Diff: https://reviews.apache.org/r/72084/diff/2/
Changes: https://reviews.apache.org/r/72084/diff/1-2/
Testing
-------
Thanks,
Karen Coppage