You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hive.apache.org by na...@apache.org on 2011/05/11 09:16:13 UTC

svn commit: r1101770 - /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/persistence/MapJoinObjectKey.java

Author: namit
Date: Wed May 11 07:16:12 2011
New Revision: 1101770

URL: http://svn.apache.org/viewvc?rev=1101770&view=rev
Log:
HIVE-2157 NPE in MapJoinObjectKey
(Yongqiang He via namit)


Modified:
    hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/persistence/MapJoinObjectKey.java

Modified: hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/persistence/MapJoinObjectKey.java
URL: http://svn.apache.org/viewvc/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/persistence/MapJoinObjectKey.java?rev=1101770&r1=1101769&r2=1101770&view=diff
==============================================================================
--- hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/persistence/MapJoinObjectKey.java (original)
+++ hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/persistence/MapJoinObjectKey.java Wed May 11 07:16:12 2011
@@ -60,6 +60,9 @@ public class MapJoinObjectKey  extends A
       if ((obj != null) && (mObjArray != null)) {
         if (obj.length == mObjArray.length) {
           for (int i = 0; i < obj.length; i++) {
+            if (obj[i] == null) {
+              return mObjArray[i] == null;
+            }
             if (!obj[i].equals(mObjArray[i])) {
               return false;
             }