----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29611/#review66930 -----------------------------------------------------------
Ship it! Ship It! - John Pullokkaran On Jan. 6, 2015, 12:50 a.m., Ashutosh Chauhan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29611/ > ----------------------------------------------------------- > > (Updated Jan. 6, 2015, 12:50 a.m.) > > > 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 > >
