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

(Updated March 27, 2017, 8 a.m.)


Review request for hive and Sergio Pena.


Changes
-------

Removed empty line from DateUtils.


Repository: hive-git


Description
-------

HIVE-16231: Parquet timestamp may be stored differently since HIVE-12767


Diffs (updated)
-----

  
ql/src/java/org/apache/hadoop/hive/ql/io/parquet/MapredParquetOutputFormat.java 
26f1e75c7d659a634cd4eef3a0cb8e886b22722f 
  ql/src/java/org/apache/hadoop/hive/ql/io/parquet/ParquetRecordReaderBase.java 
8e33b7d437894b33b35f32913a3bc02f2a849ce3 
  ql/src/java/org/apache/hadoop/hive/ql/io/parquet/timestamp/NanoTimeUtils.java 
5dc808800290f3274afbdff12134ac34387a746b 
  
ql/src/test/org/apache/hadoop/hive/ql/io/parquet/timestamp/TestNanoTimeUtils.java
 37cf0e2d74589cfa97fa24c9d2d8d00ea62390ee 
  ql/src/test/queries/clientpositive/parquet_int96_timestamp.q 
5de2c3f1244b8340b97eb0547fe66e52d80fb065 


Diff: https://reviews.apache.org/r/57728/diff/4/

Changes: https://reviews.apache.org/r/57728/diff/3-4/


Testing
-------

Tested loading timestamps from a parquet file written by spark.


Thanks,

Barna Zsombor Klara

Reply via email to