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

Posted to dev@hive.apache.org by Deepak Jaiswal <dj...@hortonworks.com> on 2017/06/23 03:16:42 UTC

Review Request 60382: HIVE-16947

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

Review request for hive and Jason Dere.


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


Repository: hive-git


Description
-------

Typically a semijoin branch and a mapjoin may create a cycle when on same operator tree. This is already handled, however, a semijoin branch can serve more than one filters and the cycle detection logic currently only handles the 1st one causing cycles preventing the queries from running.


Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/optimizer/ConvertJoinMapJoin.java 98fec77010 


Diff: https://reviews.apache.org/r/60382/diff/1/


Testing
-------


Thanks,

Deepak Jaiswal

Re: Review Request 60382: HIVE-16947

Posted by Deepak Jaiswal <dj...@hortonworks.com>.

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

(Updated June 27, 2017, 8:38 p.m.)


Review request for hive and Jason Dere.


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


Repository: hive-git


Description
-------

Typically a semijoin branch and a mapjoin may create a cycle when on same operator tree. This is already handled, however, a semijoin branch can serve more than one filters and the cycle detection logic currently only handles the 1st one causing cycles preventing the queries from running.


Diffs (updated)
-----

  ql/src/java/org/apache/hadoop/hive/ql/optimizer/ConvertJoinMapJoin.java 98fec77010 


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

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


Testing
-------


Thanks,

Deepak Jaiswal