hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Harish Butani" <rhbut...@gmail.com>
Subject Re: Review Request 14953: Pushdown join conditions
Date Tue, 29 Oct 2013 21:19:08 GMT

-----------------------------------------------------------
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


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