hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Harish Butani" <rhbut...@gmail.com>
Subject Review Request 14953: Pushdown join conditions
Date Fri, 25 Oct 2013 22:54:43 GMT

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

Review request for hive and Ashutosh Chauhan.


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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message