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 7BD1818325 for ; Mon, 28 Dec 2015 22:41:15 +0000 (UTC) Received: (qmail 80632 invoked by uid 500); 28 Dec 2015 22:41:15 -0000 Delivered-To: apmail-asterixdb-notifications-archive@asterixdb.apache.org Received: (qmail 80600 invoked by uid 500); 28 Dec 2015 22:41:15 -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 80591 invoked by uid 99); 28 Dec 2015 22:41:15 -0000 Received: from Unknown (HELO spamd3-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 28 Dec 2015 22:41:15 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd3-us-west.apache.org (ASF Mail Server at spamd3-us-west.apache.org) with ESMTP id D47EE180186 for ; Mon, 28 Dec 2015 22:41:14 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd3-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: 0.92 X-Spam-Level: X-Spam-Status: No, score=0.92 tagged_above=-999 required=6.31 tests=[SPF_FAIL=0.919, URIBL_BLOCKED=0.001] autolearn=disabled Received: from mx1-us-west.apache.org ([10.40.0.8]) by localhost (spamd3-us-west.apache.org [10.40.0.10]) (amavisd-new, port 10024) with ESMTP id 2SPwC3O4lLnH for ; Mon, 28 Dec 2015 22:41:06 +0000 (UTC) Received: from unhygienix.ics.uci.edu (unhygienix.ics.uci.edu [128.195.14.130]) by mx1-us-west.apache.org (ASF Mail Server at mx1-us-west.apache.org) with ESMTP id CB29E2049E for ; Mon, 28 Dec 2015 22:41:06 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by unhygienix.ics.uci.edu (Postfix) with ESMTP id 16CF024073E; Mon, 28 Dec 2015 14:37:25 -0800 (PST) Date: Mon, 28 Dec 2015 14:37:24 -0800 From: "Yingyi Bu (Code Review)" To: Steven Jacobs , Jenkins CC: Ildar Absalyamov , Till Westmann , Preston Carman Reply-To: buyingyi@gmail.com X-Gerrit-MessageType: newpatchset Subject: Change in asterixdb[master]: Fix for ASTERIXDB-1018, ASTERIXDB-1017, ASTERIXDB-1019, ASTE... X-Gerrit-Change-Id: Ifc1f844ac6e06e00da3f1618235b6ce9bfd48e9f X-Gerrit-ChangeURL: X-Gerrit-Commit: 060b19cc582da608cd7494a136db132fdab30b5c 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: <20151228223725.16CF024073E@unhygienix.ics.uci.edu> Hello Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/489 to look at the new patch set (#16). Change subject: Fix for ASTERIXDB-1018, ASTERIXDB-1017, ASTERIXDB-1019, ASTERIXDB-1020, ASTERIXDB-1029, ASTERIXDB-1030, ASTERIXDB-1034 ...................................................................... Fix for ASTERIXDB-1018, ASTERIXDB-1017, ASTERIXDB-1019, ASTERIXDB-1020, ASTERIXDB-1029, ASTERIXDB-1030, ASTERIXDB-1034 1. Inline SubplanOperator input and hence remove SubplanOperator. 2. Use boolean return from initFromSubTree to confirm that the result should actually be used in introducejoinaccessmethodrule Prevent accidental attempted cast of null variable in abstractintroduceaccessmethodrule. Change-Id: Ifc1f844ac6e06e00da3f1618235b6ce9bfd48e9f --- D asterix-algebra/src/main/java/org/apache/asterix/algebra/base/LogicalExpressionDeepCopyVisitor.java D asterix-algebra/src/main/java/org/apache/asterix/algebra/base/LogicalOperatorDeepCopyVisitor.java 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 M asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/PushGroupByThroughProduct.java M asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/UnnestToDataScanRule.java M asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/AbstractIntroduceAccessMethodRule.java M asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/IntroduceJoinAccessMethodRule.java M asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/InvertedIndexAccessMethod.java R asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/subplan/AsterixMoveFreeVariableOperatorOutOfSubplanRule.java A asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/subplan/InlineSubplanInputForNestedTupleSourceRule.java M asterix-algebra/src/main/java/org/apache/asterix/translator/LangExpressionToPlanTranslator.java A asterix-app/src/test/resources/optimizerts/queries/udfs/query-ASTERIXDB-1017-2.aql A asterix-app/src/test/resources/optimizerts/queries/udfs/query-ASTERIXDB-1017-recursive-2.aql A asterix-app/src/test/resources/optimizerts/queries/udfs/query-ASTERIXDB-1017-recursive.aql A asterix-app/src/test/resources/optimizerts/queries/udfs/query-ASTERIXDB-1017.aql A asterix-app/src/test/resources/optimizerts/queries/udfs/query-ASTERIXDB-1018.aql A asterix-app/src/test/resources/optimizerts/queries/udfs/query-ASTERIXDB-1019.aql A asterix-app/src/test/resources/optimizerts/queries/udfs/query-ASTERIXDB-1020.aql A asterix-app/src/test/resources/optimizerts/queries/udfs/query-ASTERIXDB-1029.aql A asterix-app/src/test/resources/optimizerts/queries/udfs/query-ASTERIXDB-1029_2.aql A asterix-app/src/test/resources/optimizerts/queries/udfs/query-ASTERIXDB-1030_1034.aql A asterix-app/src/test/resources/optimizerts/queries/udfs/query-ASTERIXDB-1030_1034_2.aql A asterix-app/src/test/resources/optimizerts/results/udfs/query-ASTERIXDB-1017-2.plan A asterix-app/src/test/resources/optimizerts/results/udfs/query-ASTERIXDB-1017-recursive-2.plan A asterix-app/src/test/resources/optimizerts/results/udfs/query-ASTERIXDB-1017-recursive.plan A asterix-app/src/test/resources/optimizerts/results/udfs/query-ASTERIXDB-1017.plan A asterix-app/src/test/resources/optimizerts/results/udfs/query-ASTERIXDB-1018.plan A asterix-app/src/test/resources/optimizerts/results/udfs/query-ASTERIXDB-1019.plan A asterix-app/src/test/resources/optimizerts/results/udfs/query-ASTERIXDB-1020.plan A asterix-app/src/test/resources/optimizerts/results/udfs/query-ASTERIXDB-1029.plan A asterix-app/src/test/resources/optimizerts/results/udfs/query-ASTERIXDB-1029_2.plan A asterix-app/src/test/resources/optimizerts/results/udfs/query-ASTERIXDB-1030_1034.plan A asterix-app/src/test/resources/optimizerts/results/udfs/query-ASTERIXDB-1030_1034_2.plan M asterix-om/src/main/java/org/apache/asterix/om/functions/AsterixBuiltinFunctions.java 35 files changed, 2,100 insertions(+), 738 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/89/489/16 -- To view, visit https://asterix-gerrit.ics.uci.edu/489 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ifc1f844ac6e06e00da3f1618235b6ce9bfd48e9f Gerrit-PatchSet: 16 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Steven Jacobs Gerrit-Reviewer: Ildar Absalyamov Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Preston Carman Gerrit-Reviewer: Steven Jacobs Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: Yingyi Bu