----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22174/ -----------------------------------------------------------
(Updated June 5, 2014, 7:33 a.m.) Review request for hive, Brock Noland, justin coffey, and Xuefu Zhang. Changes ------- Cache the Calendar in thread-local as suggested for performance. Bugs: HIVE-6394 https://issues.apache.org/jira/browse/HIVE-6394 Repository: hive-git Description ------- This uses the Jodd library to convert java.sql.Timestamp type used by Hive into the {julian-day:nanos} format expected by parquet, and vice-versa. Diffs (updated) ----- data/files/parquet_types.txt 0be390b pom.xml 4bb8880 ql/pom.xml 13c477a ql/src/java/org/apache/hadoop/hive/ql/io/parquet/convert/ETypeConverter.java 4da0d30 ql/src/java/org/apache/hadoop/hive/ql/io/parquet/convert/HiveSchemaConverter.java 29f7e11 ql/src/java/org/apache/hadoop/hive/ql/io/parquet/serde/ArrayWritableObjectInspector.java 57161d8 ql/src/java/org/apache/hadoop/hive/ql/io/parquet/serde/ParquetHiveSerDe.java fb2f5a8 ql/src/java/org/apache/hadoop/hive/ql/io/parquet/utils/NanoTimeUtils.java PRE-CREATION ql/src/java/org/apache/hadoop/hive/ql/io/parquet/write/DataWritableWriter.java 3490061 ql/src/test/org/apache/hadoop/hive/ql/io/parquet/serde/TestParquetTimestampUtils.java PRE-CREATION ql/src/test/queries/clientpositive/parquet_types.q 5d6333c ql/src/test/results/clientpositive/parquet_types.q.out c23f7f1 Diff: https://reviews.apache.org/r/22174/diff/ Testing ------- Unit tests the new libraries, and also added timestamp data in the "parquet_types" q-test. Thanks, Szehon Ho