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

(Updated June 10, 2014, 4:36 p.m.)


Review request for hive, Brock Noland, justin coffey, and Xuefu Zhang.


Changes
-------

Removing a negative test asserting that timestamp is not supported.


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 2b91846 
  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/clientnegative/parquet_timestamp.q 4ef36fa 
  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

Reply via email to