----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28933/#review65334 -----------------------------------------------------------
Ship it! Ship It! - Ryan Blue On Dec. 15, 2014, 7:40 p.m., cheng xu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28933/ > ----------------------------------------------------------- > > (Updated Dec. 15, 2014, 7:40 p.m.) > > > Review request for hive. > > > Repository: hive-git > > > Description > ------- > > The patch includes: > 1.add timestamp support for AvroSerde > 2.add related test cases > > > Diffs > ----- > > 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 > >