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

(Updated Dec. 22, 2017, 11:55 p.m.)


Review request for hive and Matt McCline.


Changes
-------

Added the similar logic to VectorMapJoinFastLongHashTable class as well.


Bugs: HIVE-18335
    https://issues.apache.org/jira/browse/HIVE-18335


Repository: hive-git


Description
-------

VectorMapJoinFastBytesHashTable.allocateBucketArray() needs to check bounds 
before allocating array.

Added a unit test.


Diffs (updated)
-----

  
ql/src/java/org/apache/hadoop/hive/ql/exec/vector/mapjoin/fast/VectorMapJoinFastBytesHashTable.java
 89087e1450 
  
ql/src/java/org/apache/hadoop/hive/ql/exec/vector/mapjoin/fast/VectorMapJoinFastLongHashTable.java
 f610653640 
  
ql/src/test/org/apache/hadoop/hive/ql/exec/vector/mapjoin/fast/TestVectorMapJoinFastBytesHashMap.java
 905180431f 
  
ql/src/test/org/apache/hadoop/hive/ql/exec/vector/mapjoin/fast/TestVectorMapJoinFastLongHashMap.java
 315a54dde5 


Diff: https://reviews.apache.org/r/64819/diff/2/

Changes: https://reviews.apache.org/r/64819/diff/1-2/


Testing
-------


Thanks,

Deepak Jaiswal

Reply via email to