----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57728/ -----------------------------------------------------------
(Updated March 24, 2017, 9:56 a.m.) Review request for hive and Sergio Pena. Changes ------- Move the time zone checking utility method into NanotTimeUtils. Repository: hive-git Description ------- HIVE-16231: Parquet timestamp may be stored differently since HIVE-12767 Diffs (updated) ----- common/src/java/org/apache/hive/common/util/DateUtils.java a1068ecce94e9ff1ae78008a0d8c6d67ca4f2690 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/3/ Changes: https://reviews.apache.org/r/57728/diff/2-3/ Testing ------- Tested loading timestamps from a parquet file written by spark. Thanks, Barna Zsombor Klara
