hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sean Busbey" <s...@manvsbeard.com>
Subject Re: Review Request: HIVE-3525
Date Thu, 04 Oct 2012 03:11:44 GMT

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

(Updated Oct. 4, 2012, 3:11 a.m.)


Review request for hive.


Changes
-------

Now includes a proposed fix, changing internal Hashtable use to HashMap.


Summary (updated)
-----------------

HIVE-3525


Description (updated)
-------

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 1393805 
  /trunk/serde/src/test/org/apache/hadoop/hive/serde2/avro/TestAvroDeserializer.java 1393805

  /trunk/serde/src/test/org/apache/hadoop/hive/serde2/avro/TestAvroObjectInspectorGenerator.java
1393805 
  /trunk/serde/src/test/org/apache/hadoop/hive/serde2/avro/TestAvroSerializer.java 1393805


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


Testing (updated)
-------

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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message