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 7CCCD176D6 for ; Tue, 4 Nov 2014 19:12:04 +0000 (UTC) Received: (qmail 90091 invoked by uid 500); 4 Nov 2014 19:12:04 -0000 Delivered-To: apmail-hive-dev-archive@hive.apache.org Received: (qmail 90015 invoked by uid 500); 4 Nov 2014 19:12:04 -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 89998 invoked by uid 99); 4 Nov 2014 19:12:03 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 04 Nov 2014 19:12:03 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id 642B51DFB5E; Tue, 4 Nov 2014 19:12:07 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============3396213477601119012==" MIME-Version: 1.0 Subject: Re: Review Request 27401: HIVE-8636 CBO: split cbo_correctness test From: "Sergey Shelukhin" To: "John Pullokkaran" , "Ashutosh Chauhan" Cc: "Sergey Shelukhin" , "hive" Date: Tue, 04 Nov 2014 19:12:07 -0000 Message-ID: <20141104191207.10454.87394@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org Auto-Submitted: auto-generated Sender: "Sergey Shelukhin" X-ReviewGroup: hive X-ReviewRequest-URL: https://reviews.apache.org/r/27401/ X-Sender: "Sergey Shelukhin" References: <20141031010155.7138.94857@reviews.apache.org> In-Reply-To: <20141031010155.7138.94857@reviews.apache.org> Reply-To: "Sergey Shelukhin" X-ReviewRequest-Repository: hive-git --===============3396213477601119012== MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit > On Oct. 31, 2014, 1:01 a.m., John Pullokkaran wrote: > > itests/src/test/resources/testconfiguration.properties, line 65 > > > > > > Why don't we put all CBO subqueries in to a single test? seems it will be too big - Sergey ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27401/#review59290 ----------------------------------------------------------- On Oct. 30, 2014, 11:18 p.m., Sergey Shelukhin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/27401/ > ----------------------------------------------------------- > > (Updated Oct. 30, 2014, 11:18 p.m.) > > > Review request for hive, Ashutosh Chauhan and John Pullokkaran. > > > Repository: hive-git > > > Description > ------- > > See JIRA > > > Diffs > ----- > > data/scripts/q_test_cleanup.sql 8ec0f9f > data/scripts/q_test_init.sql 7484f0c > itests/src/test/resources/testconfiguration.properties 2c84a36 > pom.xml bd74830 > ql/src/test/queries/clientpositive/cbo_correctness.q bb328f6 > ql/src/test/queries/clientpositive/cbo_gby.q PRE-CREATION > ql/src/test/queries/clientpositive/cbo_gby_empty.q PRE-CREATION > ql/src/test/queries/clientpositive/cbo_join.q PRE-CREATION > ql/src/test/queries/clientpositive/cbo_limit.q PRE-CREATION > ql/src/test/queries/clientpositive/cbo_semijoin.q PRE-CREATION > ql/src/test/queries/clientpositive/cbo_simple_select.q PRE-CREATION > ql/src/test/queries/clientpositive/cbo_stats.q PRE-CREATION > ql/src/test/queries/clientpositive/cbo_subq_exists.q PRE-CREATION > ql/src/test/queries/clientpositive/cbo_subq_in.q PRE-CREATION > ql/src/test/queries/clientpositive/cbo_subq_not_in.q PRE-CREATION > ql/src/test/queries/clientpositive/cbo_udf_udaf.q PRE-CREATION > ql/src/test/queries/clientpositive/cbo_union.q PRE-CREATION > ql/src/test/queries/clientpositive/cbo_views.q PRE-CREATION > ql/src/test/queries/clientpositive/cbo_windowing.q PRE-CREATION > ql/src/test/results/clientpositive/cbo_correctness.q.out d98cb5b > ql/src/test/results/clientpositive/cbo_gby.q.out PRE-CREATION > ql/src/test/results/clientpositive/cbo_gby_empty.q.out PRE-CREATION > ql/src/test/results/clientpositive/cbo_join.q.out PRE-CREATION > ql/src/test/results/clientpositive/cbo_limit.q.out PRE-CREATION > ql/src/test/results/clientpositive/cbo_semijoin.q.out PRE-CREATION > ql/src/test/results/clientpositive/cbo_simple_select.q.out PRE-CREATION > ql/src/test/results/clientpositive/cbo_stats.q.out PRE-CREATION > ql/src/test/results/clientpositive/cbo_subq_exists.q.out PRE-CREATION > ql/src/test/results/clientpositive/cbo_subq_in.q.out PRE-CREATION > ql/src/test/results/clientpositive/cbo_subq_not_in.q.out PRE-CREATION > ql/src/test/results/clientpositive/cbo_udf_udaf.q.out PRE-CREATION > ql/src/test/results/clientpositive/cbo_union.q.out PRE-CREATION > ql/src/test/results/clientpositive/cbo_views.q.out PRE-CREATION > ql/src/test/results/clientpositive/cbo_windowing.q.out PRE-CREATION > ql/src/test/results/clientpositive/tez/cbo_correctness.q.out 2103d08 > ql/src/test/results/clientpositive/tez/cbo_gby.q.out PRE-CREATION > ql/src/test/results/clientpositive/tez/cbo_gby_empty.q.out PRE-CREATION > ql/src/test/results/clientpositive/tez/cbo_join.q.out PRE-CREATION > ql/src/test/results/clientpositive/tez/cbo_limit.q.out PRE-CREATION > ql/src/test/results/clientpositive/tez/cbo_semijoin.q.out PRE-CREATION > ql/src/test/results/clientpositive/tez/cbo_simple_select.q.out PRE-CREATION > ql/src/test/results/clientpositive/tez/cbo_stats.q.out PRE-CREATION > ql/src/test/results/clientpositive/tez/cbo_subq_exists.q.out PRE-CREATION > ql/src/test/results/clientpositive/tez/cbo_subq_in.q.out PRE-CREATION > ql/src/test/results/clientpositive/tez/cbo_subq_not_in.q.out PRE-CREATION > ql/src/test/results/clientpositive/tez/cbo_udf_udaf.q.out PRE-CREATION > ql/src/test/results/clientpositive/tez/cbo_union.q.out PRE-CREATION > ql/src/test/results/clientpositive/tez/cbo_views.q.out PRE-CREATION > ql/src/test/results/clientpositive/tez/cbo_windowing.q.out PRE-CREATION > > Diff: https://reviews.apache.org/r/27401/diff/ > > > Testing > ------- > > > Thanks, > > Sergey Shelukhin > > --===============3396213477601119012==--