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

(Updated Dec. 16, 2014, 3:40 a.m.)


Review request for hive.


Changes
-------

Summary for the updates:
1. change the type from timestamp to timestamp-millis
2. change the data's precision from nanos to millis
3. clean code


Repository: hive-git


Description
-------

The patch includes:
1.add timestamp support for AvroSerde 
2.add related test cases


Diffs (updated)
-----

  data/files/avro_timestamp.txt PRE-CREATION 
  ql/src/test/queries/clientpositive/avro_timestamp.q PRE-CREATION 
  ql/src/test/results/clientpositive/avro_timestamp.q.out PRE-CREATION 
  serde/src/java/org/apache/hadoop/hive/serde2/avro/AvroDeserializer.java 
07c5ecf 
  serde/src/java/org/apache/hadoop/hive/serde2/avro/AvroSerDe.java 7639a2b 
  serde/src/java/org/apache/hadoop/hive/serde2/avro/AvroSerializer.java c8eac89 
  serde/src/java/org/apache/hadoop/hive/serde2/avro/SchemaToTypeInfo.java 
c84b1a0 
  serde/src/java/org/apache/hadoop/hive/serde2/avro/TypeInfoToSchema.java 
8cb2dc3 
  serde/src/test/org/apache/hadoop/hive/serde2/avro/TestTypeInfoToSchema.java 
cd5a0fa 

Diff: https://reviews.apache.org/r/28933/diff/


Testing
-------

Test passed for added cases


Thanks,

cheng xu

Reply via email to