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

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


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

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

+
Harish Butani 2013-10-29, 21:19
+
Ashutosh Chauhan 2013-10-30, 19:43
+
Harish Butani 2013-10-30, 20:21
+
Ashutosh Chauhan 2013-10-31, 00:34
+
Harish Butani 2013-10-31, 01:15
+
Ashutosh Chauhan 2013-10-31, 14:07