hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ashutosh Chauhan" <hashut...@apache.org>
Subject Review Request 29611: Add ProjectRemove rule on CBO path
Date Tue, 06 Jan 2015 00:50:45 GMT

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

Review request for hive and John Pullokkaran.


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


Repository: hive-git


Description
-------

Add ProjectRemove rule on CBO path. All plans changed for better, with couple of tests generating
better plans doing multiple joins within same MR task (as oppose to multiple earlier).


Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java 5d72e15 
  ql/src/test/results/clientpositive/auto_join16.q.out 10e0161 
  ql/src/test/results/clientpositive/auto_join17.q.out 3ec3b19 
  ql/src/test/results/clientpositive/auto_join19.q.out 141111f 
  ql/src/test/results/clientpositive/auto_join2.q.out 09111d9 
  ql/src/test/results/clientpositive/auto_join22.q.out faa641e 
  ql/src/test/results/clientpositive/auto_join27.q.out d82cdb4 
  ql/src/test/results/clientpositive/auto_join3.q.out a854e1e 
  ql/src/test/results/clientpositive/auto_join33.q.out 4838f75 
  ql/src/test/results/clientpositive/auto_join7.q.out 946a837 
  ql/src/test/results/clientpositive/auto_join8.q.out 07eab3b 
  ql/src/test/results/clientpositive/auto_join9.q.out 3eb3d18 
  ql/src/test/results/clientpositive/auto_join_nulls.q.out 3df919c 
  ql/src/test/results/clientpositive/auto_join_without_localtask.q.out 1ad03f4 
  ql/src/test/results/clientpositive/correlationoptimizer3.q.out bb8a3c5 
  ql/src/test/results/clientpositive/correlationoptimizer6.q.out f0cf64c 
  ql/src/test/results/clientpositive/correlationoptimizer7.q.out eca6f27 
  ql/src/test/results/clientpositive/cross_product_check_2.q.out e4e368b 
  ql/src/test/results/clientpositive/explain_logical.q.out d28c685 
  ql/src/test/results/clientpositive/input23.q.out 0c03730 
  ql/src/test/results/clientpositive/join17.q.out 4315416 
  ql/src/test/results/clientpositive/join32_lessSize.q.out b1fc823 
  ql/src/test/results/clientpositive/join34.q.out e053e44 
  ql/src/test/results/clientpositive/join35.q.out 76de7bf 
  ql/src/test/results/clientpositive/join7.q.out 219714e 
  ql/src/test/results/clientpositive/join9.q.out ae46952 
  ql/src/test/results/clientpositive/louter_join_ppr.q.out 5394736 
  ql/src/test/results/clientpositive/mapjoin_mapjoin.q.out 3c68c40 
  ql/src/test/results/clientpositive/metadataonly1.q.out c0e7b17 
  ql/src/test/results/clientpositive/multiMapJoin2.q.out 5deac50 
  ql/src/test/results/clientpositive/ppd_join_filter.q.out 12245bf 
  ql/src/test/results/clientpositive/ppd_union_view.q.out 1729464 
  ql/src/test/results/clientpositive/ppd_vc.q.out 82c6aa6 
  ql/src/test/results/clientpositive/router_join_ppr.q.out 8e16942 
  ql/src/test/results/clientpositive/subquery_views.q.out 9d29854 
  ql/src/test/results/clientpositive/tez/cross_join.q.out f4efd8f 
  ql/src/test/results/clientpositive/tez/cross_product_check_1.q.out 05e9f43 
  ql/src/test/results/clientpositive/tez/cross_product_check_2.q.out cfd56a3 
  ql/src/test/results/clientpositive/vector_left_outer_join.q.out deb0daa 
  ql/src/test/results/clientpositive/vectorized_mapjoin.q.out e6cfae7 
  ql/src/test/results/clientpositive/vectorized_nested_mapjoin.q.out 74983ee 

Diff: https://reviews.apache.org/r/29611/diff/


Testing
-------

Existing tests.


Thanks,

Ashutosh Chauhan


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