Return-Path: X-Original-To: apmail-hive-dev-archive@www.apache.org Delivered-To: apmail-hive-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 83D7E179B5 for ; Tue, 6 Jan 2015 00:50:48 +0000 (UTC) Received: (qmail 55297 invoked by uid 500); 6 Jan 2015 00:50:49 -0000 Delivered-To: apmail-hive-dev-archive@hive.apache.org Received: (qmail 55232 invoked by uid 500); 6 Jan 2015 00:50:49 -0000 Mailing-List: contact dev-help@hive.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@hive.apache.org Delivered-To: mailing list dev@hive.apache.org Received: (qmail 55200 invoked by uid 99); 6 Jan 2015 00:50:47 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 06 Jan 2015 00:50:47 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id 696D51CABF7; Tue, 6 Jan 2015 00:50:45 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============2415099968964135709==" MIME-Version: 1.0 Subject: Review Request 29611: Add ProjectRemove rule on CBO path From: "Ashutosh Chauhan" To: "John Pullokkaran" Cc: "Ashutosh Chauhan" , "hive" Date: Tue, 06 Jan 2015 00:50:45 -0000 Message-ID: <20150106005045.26560.66160@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org Auto-Submitted: auto-generated Sender: "Ashutosh Chauhan" X-ReviewGroup: hive X-ReviewRequest-URL: https://reviews.apache.org/r/29611/ X-Sender: "Ashutosh Chauhan" Reply-To: "Ashutosh Chauhan" X-ReviewRequest-Repository: hive-git --===============2415099968964135709== MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit ----------------------------------------------------------- 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 --===============2415099968964135709==--