----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7430/ -----------------------------------------------------------
(Updated Oct. 4, 2012, 1:56 p.m.) Review request for hive. Changes ------- Last update missed the changes to AvroSerializer Description ------- Changes Avro SerDe to use HashMap when copying out the Avro Map<Utf8, Object> to Map<String, Object>. fixes HIVE-3525. Diffs (updated) ----- /trunk/serde/src/java/org/apache/hadoop/hive/serde2/avro/AvroDeserializer.java 1394076 /trunk/serde/src/java/org/apache/hadoop/hive/serde2/avro/AvroSerializer.java 1394076 /trunk/serde/src/test/org/apache/hadoop/hive/serde2/avro/TestAvroDeserializer.java 1394076 /trunk/serde/src/test/org/apache/hadoop/hive/serde2/avro/TestAvroObjectInspectorGenerator.java 1394076 /trunk/serde/src/test/org/apache/hadoop/hive/serde2/avro/TestAvroSerializer.java 1394076 Diff: https://reviews.apache.org/r/7430/diff/ Testing ------- Includes unit tests for * AvroObjectInspectorGenerator to verify that the Nullable value type is presented as just the non-null type. * AvroDeserializer to verify that Maps with null are properly handled * AvroSerializer to verify that Maps with null can round trip. Thanks, Sean Busbey
