You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by ha...@apache.org on 2014/03/03 19:04:27 UTC

svn commit: r1573644 - /hive/trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/objectinspector/LazyBinaryMapObjectInspector.java

Author: hashutosh
Date: Mon Mar  3 18:04:26 2014
New Revision: 1573644

URL: http://svn.apache.org/r1573644
Log:
HIVE-6389 : LazyBinaryColumnarSerDe-based RCFile tables break when looking up elements in null-maps. (Mithun Radhakrishnan via Ashutosh Chauhan)

Modified:
    hive/trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/objectinspector/LazyBinaryMapObjectInspector.java

Modified: hive/trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/objectinspector/LazyBinaryMapObjectInspector.java
URL: http://svn.apache.org/viewvc/hive/trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/objectinspector/LazyBinaryMapObjectInspector.java?rev=1573644&r1=1573643&r2=1573644&view=diff
==============================================================================
--- hive/trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/objectinspector/LazyBinaryMapObjectInspector.java (original)
+++ hive/trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/objectinspector/LazyBinaryMapObjectInspector.java Mon Mar  3 18:04:26 2014
@@ -56,8 +56,8 @@ public class LazyBinaryMapObjectInspecto
 
   @Override
   public Object getMapValueElement(Object data, Object key) {
-    if (data == null) {
-      return -1;
+    if (data == null || key == null) {
+      return null;
     }
     return ((LazyBinaryMap) data).getMapValueElement(key);
   }