-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61087/
-----------------------------------------------------------
(Updated July 27, 2017, 9:25 p.m.)
Review request for hive, Gopal V, Jason Dere, and Sergey Shelukhin.
Changes
-------
Fixed the assert introduced in last rev. to compare the path values instead of
comparing the path objects.
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)
-----
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/results/clientpositive/llap/llap_smb.q.out 87b33db805
Diff: https://reviews.apache.org/r/61087/diff/5/
Changes: https://reviews.apache.org/r/61087/diff/4-5/
Testing
-------
Added a new test.
Thanks,
Deepak Jaiswal
