Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Hive >> mail # dev >> Review Request 14953: Pushdown join conditions


Copy link to this message
-
Re: Review Request 14953: Pushdown join conditions

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

(Updated Oct. 29, 2013, 9:19 p.m.)
Review request for hive, Ashutosh Chauhan and Vikram Dixit Kumaraswamy.
Bugs: hive-5556
    https://issues.apache.org/jira/browse/hive-5556
Repository: hive-git
Description
-------

Step 1 to support Alternate Join Syntax: HIVE-5555

This patch also contains fixes to merging of QBJoinTrees
Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/parse/QBJoinTree.java 9c8cac1
  ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java cf0c895
  ql/src/test/org/apache/hadoop/hive/ql/parse/TestQBJoinTreeApplyPredicate.java PRE-CREATION
  ql/src/test/queries/clientpositive/join_cond_pushdown_1.q PRE-CREATION
  ql/src/test/queries/clientpositive/join_cond_pushdown_2.q PRE-CREATION
  ql/src/test/results/clientpositive/auto_sortmerge_join_12.q.out 865627b
  ql/src/test/results/clientpositive/join_cond_pushdown_1.q.out PRE-CREATION
  ql/src/test/results/clientpositive/join_cond_pushdown_2.q.out PRE-CREATION

Diff: https://reviews.apache.org/r/14953/diff/
Testing
-------

ran all join .q files
added join_cond_pushdown_1.q, join_cond_pushdown_2.q .q tests
added TestQBJoinTreeApplyPredicate unit test to test pushdown functionality
Thanks,

Harish Butani