Return-Path: X-Original-To: apmail-asterixdb-notifications-archive@minotaur.apache.org Delivered-To: apmail-asterixdb-notifications-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id B48E4187EA for ; Sat, 13 Feb 2016 01:50:02 +0000 (UTC) Received: (qmail 54108 invoked by uid 500); 13 Feb 2016 01:50:02 -0000 Delivered-To: apmail-asterixdb-notifications-archive@asterixdb.apache.org Received: (qmail 54068 invoked by uid 500); 13 Feb 2016 01:50:02 -0000 Mailing-List: contact notifications-help@asterixdb.incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@asterixdb.incubator.apache.org Delivered-To: mailing list notifications@asterixdb.incubator.apache.org Received: (qmail 54053 invoked by uid 99); 13 Feb 2016 01:50:02 -0000 Received: from Unknown (HELO spamd4-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 13 Feb 2016 01:50:02 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd4-us-west.apache.org (ASF Mail Server at spamd4-us-west.apache.org) with ESMTP id 2CED0C0D87 for ; Sat, 13 Feb 2016 01:50:02 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd4-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: 0.919 X-Spam-Level: X-Spam-Status: No, score=0.919 tagged_above=-999 required=6.31 tests=[SPF_FAIL=0.919] autolearn=disabled Received: from mx1-lw-us.apache.org ([10.40.0.8]) by localhost (spamd4-us-west.apache.org [10.40.0.11]) (amavisd-new, port 10024) with ESMTP id g2h_TbJvKlST for ; Sat, 13 Feb 2016 01:50:00 +0000 (UTC) Received: from unhygienix.ics.uci.edu (unhygienix.ics.uci.edu [128.195.14.130]) by mx1-lw-us.apache.org (ASF Mail Server at mx1-lw-us.apache.org) with ESMTP id 40B5F5FC0F for ; Sat, 13 Feb 2016 01:50:00 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by unhygienix.ics.uci.edu (Postfix) with ESMTP id 4E49A241A1E; Fri, 12 Feb 2016 17:45:38 -0800 (PST) Date: Fri, 12 Feb 2016 17:45:38 -0800 From: "Yingyi Bu (Code Review)" To: Till Westmann , Jenkins CC: Jianfeng Jia , Preston Carman Reply-To: buyingyi@gmail.com X-Gerrit-MessageType: newpatchset Subject: Change in asterixdb[master]: Fix for ASTERIXDB-1205 X-Gerrit-Change-Id: Ib79a899ec4b35ece6f8f9f59b80126fed0c48851 X-Gerrit-ChangeURL: X-Gerrit-Commit: df5e4ddd9f3f38baec51ceb4766f80484b9ed27e In-Reply-To: References: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Content-Disposition: inline User-Agent: Gerrit/2.8.4 Message-Id: <20160213014538.4E49A241A1E@unhygienix.ics.uci.edu> Hello Till Westmann, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/629 to look at the new patch set (#9). Change subject: Fix for ASTERIXDB-1205 ...................................................................... Fix for ASTERIXDB-1205 -fixed the AqlExpressionToPlanTranslator for union expression; -removed IntroduceUnionRule; -added regression tests. Change-Id: Ib79a899ec4b35ece6f8f9f59b80126fed0c48851 --- M asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/RuleCollections.java M asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/FuzzyJoinRule.java D asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/IntroduceUnionRule.java M asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/InvertedIndexAccessMethod.java M asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/subplan/InlineAllNtsInSubplanVisitor.java M asterix-algebra/src/main/java/org/apache/asterix/translator/AqlExpressionToPlanTranslator.java M asterix-algebra/src/main/java/org/apache/asterix/translator/LangExpressionToPlanTranslator.java M asterix-algebra/src/main/java/org/apache/asterix/translator/SqlppExpressionToPlanTranslator.java M asterix-algebra/src/main/java/org/apache/asterix/translator/TranslationContext.java A asterix-app/src/test/resources/optimizerts/queries/union/query-ASTERIXDB-1205-2.aql A asterix-app/src/test/resources/optimizerts/queries/union/query-ASTERIXDB-1205-3.aql A asterix-app/src/test/resources/optimizerts/queries/union/query-ASTERIXDB-1205-4.aql A asterix-app/src/test/resources/optimizerts/queries/union/query-ASTERIXDB-1205.aql A asterix-app/src/test/resources/optimizerts/queries/union/union_dataset.aql A asterix-app/src/test/resources/optimizerts/queries/union/union_dataset2.aql A asterix-app/src/test/resources/optimizerts/queries/union/union_dataset3.aql A asterix-app/src/test/resources/optimizerts/queries/union/union_query.aql A asterix-app/src/test/resources/optimizerts/results/union/query-ASTERIXDB-1205-2.plan A asterix-app/src/test/resources/optimizerts/results/union/query-ASTERIXDB-1205-3.plan A asterix-app/src/test/resources/optimizerts/results/union/query-ASTERIXDB-1205-4.plan A asterix-app/src/test/resources/optimizerts/results/union/query-ASTERIXDB-1205.plan A asterix-app/src/test/resources/optimizerts/results/union/union_dataset.plan A asterix-app/src/test/resources/optimizerts/results/union/union_dataset2.plan A asterix-app/src/test/resources/optimizerts/results/union/union_dataset3.plan A asterix-app/src/test/resources/optimizerts/results/union/union_query.plan A asterix-app/src/test/resources/runtimets/queries/union/query-ASTERIXDB-1205-2/query-ASTERIXDB-1205.1.ddl.aql A asterix-app/src/test/resources/runtimets/queries/union/query-ASTERIXDB-1205-2/query-ASTERIXDB-1205.2.update.aql A asterix-app/src/test/resources/runtimets/queries/union/query-ASTERIXDB-1205-2/query-ASTERIXDB-1205.3.query.aql A asterix-app/src/test/resources/runtimets/queries/union/query-ASTERIXDB-1205-3/query-ASTERIXDB-1205.1.ddl.aql A asterix-app/src/test/resources/runtimets/queries/union/query-ASTERIXDB-1205-3/query-ASTERIXDB-1205.2.update.aql A asterix-app/src/test/resources/runtimets/queries/union/query-ASTERIXDB-1205-3/query-ASTERIXDB-1205.3.query.aql A asterix-app/src/test/resources/runtimets/queries/union/query-ASTERIXDB-1205/query-ASTERIXDB-1205.1.ddl.aql A asterix-app/src/test/resources/runtimets/queries/union/query-ASTERIXDB-1205/query-ASTERIXDB-1205.2.update.aql A asterix-app/src/test/resources/runtimets/queries/union/query-ASTERIXDB-1205/query-ASTERIXDB-1205.3.query.aql A asterix-app/src/test/resources/runtimets/queries/union/query-ASTERIXDB-300/query-ASTERIXDB-300.1.ddl.aql A asterix-app/src/test/resources/runtimets/queries/union/query-ASTERIXDB-300/query-ASTERIXDB-300.2.update.aql A asterix-app/src/test/resources/runtimets/queries/union/query-ASTERIXDB-300/query-ASTERIXDB-300.3.query.aql M asterix-app/src/test/resources/runtimets/queries/union/union/union.3.query.aql A asterix-app/src/test/resources/runtimets/queries/union/union2/union.1.ddl.aql A asterix-app/src/test/resources/runtimets/queries/union/union2/union.2.update.aql A asterix-app/src/test/resources/runtimets/queries/union/union2/union.3.query.aql A asterix-app/src/test/resources/runtimets/results/union/query-ASTERIXDB-1205-2/query-ASTERIXDB-1205-2.1.adm A asterix-app/src/test/resources/runtimets/results/union/query-ASTERIXDB-1205-3/query-ASTERIXDB-1205-3.1.adm A asterix-app/src/test/resources/runtimets/results/union/query-ASTERIXDB-1205/query-ASTERIXDB-1205.1.adm A asterix-app/src/test/resources/runtimets/results/union/query-ASTERIXDB-300/query-ASTERIXDB-300.1.adm M asterix-app/src/test/resources/runtimets/results/union/union/union.1.adm A asterix-app/src/test/resources/runtimets/results/union/union2/union2.1.adm M asterix-app/src/test/resources/runtimets/testsuite.xml M asterix-om/src/main/java/org/apache/asterix/om/functions/AsterixBuiltinFunctions.java 49 files changed, 918 insertions(+), 189 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/29/629/9 -- To view, visit https://asterix-gerrit.ics.uci.edu/629 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ib79a899ec4b35ece6f8f9f59b80126fed0c48851 Gerrit-PatchSet: 9 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Yingyi Bu Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Jianfeng Jia Gerrit-Reviewer: Preston Carman Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Yingyi Bu