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