-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61087/
-----------------------------------------------------------
(Updated July 24, 2017, 9:35 p.m.)
Review request for hive, Gopal V, Jason Dere, and Sergey Shelukhin.
Changes
-------
Implemented comments from Sergey and Gopal.
Bugs: HIVE-16965
https://issues.apache.org/jira/browse/HIVE-16965
Repository: hive-git
Description
-------
Usually, in a JOIN with multiple inputs (partitions), the inputs are read
sequentially, however, incase of SMB join, the inputs are read based on key
ordering. This invalidates the current IOContext assumption that the input path
once set wont change unless the input changes.
This was resulting in incorrect partition information in results as it is
derived from the input path in IOContext.
The new logic changes the input path as and when input changes.
Diffs (updated)
-----
itests/src/test/resources/testconfiguration.properties f66e19be3e
ql/src/java/org/apache/hadoop/hive/ql/exec/tez/MapRecordSource.java
add7d08c40
ql/src/java/org/apache/hadoop/hive/ql/exec/tez/tools/KeyValueInputMerger.java
698fa7f69e
ql/src/test/queries/clientpositive/smb_join1.q PRE-CREATION
ql/src/test/results/clientpositive/llap/smb_join1.q.out PRE-CREATION
Diff: https://reviews.apache.org/r/61087/diff/2/
Changes: https://reviews.apache.org/r/61087/diff/1-2/
Testing
-------
Added a new test.
Thanks,
Deepak Jaiswal