asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Yingyi Bu (Code Review)" <do-not-re...@asterixdb.incubator.apache.org>
Subject Change in asterixdb[master]: Fix for ASTERIXDB-1205
Date Fri, 19 Feb 2016 19:57:12 GMT
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 (#15).

Change subject: Fix for ASTERIXDB-1205
......................................................................

Fix for ASTERIXDB-1205

-fixed the AqlExpressionToPlanTranslator for union expression;
-eliminate shared operator references by deep copying for the translated logical plan;
-removed IntroduceUnionRule;
-fixed SQL++ "join ... on" to be able to access variables defined by WITH bindings;
-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-1047.aql
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
M asterix-app/src/test/resources/optimizerts/results/q05_local_supplier_volume.plan
M asterix-app/src/test/resources/optimizerts/results/q08_group_by.plan
M asterix-app/src/test/resources/optimizerts/results/q09_group_by.plan
M asterix-app/src/test/resources/optimizerts/results/query-issue562.plan
M asterix-app/src/test/resources/optimizerts/results/split-materialization-above-join.plan
A asterix-app/src/test/resources/optimizerts/results/union/query-ASTERIXDB-1047.plan
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-1047/query-ASTERIXDB-1047.1.ddl.aql
A asterix-app/src/test/resources/runtimets/queries/union/query-ASTERIXDB-1047/query-ASTERIXDB-1047.2.update.aql
A asterix-app/src/test/resources/runtimets/queries/union/query-ASTERIXDB-1047/query-ASTERIXDB-1047.3.query.aql
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
M asterix-app/src/test/resources/runtimets/queries_sqlpp/custord/join_q_04/join_q_04.1.ddl.sqlpp
A asterix-app/src/test/resources/runtimets/queries_sqlpp/custord/join_q_05/join_q_05.1.ddl.sqlpp
A asterix-app/src/test/resources/runtimets/queries_sqlpp/custord/join_q_05/join_q_05.2.update.sqlpp
A asterix-app/src/test/resources/runtimets/queries_sqlpp/custord/join_q_05/join_q_05.3.query.sqlpp
A asterix-app/src/test/resources/runtimets/queries_sqlpp/custord/join_q_06/join_q_06.1.ddl.sqlpp
A asterix-app/src/test/resources/runtimets/queries_sqlpp/custord/join_q_06/join_q_06.2.update.sqlpp
A asterix-app/src/test/resources/runtimets/queries_sqlpp/custord/join_q_06/join_q_06.3.query.sqlpp
A asterix-app/src/test/resources/runtimets/queries_sqlpp/custord/join_q_07/join_q_07.1.ddl.sqlpp
A asterix-app/src/test/resources/runtimets/queries_sqlpp/custord/join_q_07/join_q_07.2.update.sqlpp
A asterix-app/src/test/resources/runtimets/queries_sqlpp/custord/join_q_07/join_q_07.3.query.sqlpp
A asterix-app/src/test/resources/runtimets/results/custord/join_q_06/join_q_06.1.adm
A asterix-app/src/test/resources/runtimets/results/union/query-ASTERIXDB-1047/query-ASTERIXDB-1047.1.adm
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/results_parser_sqlpp/custord/join_q_04/join_q_04.1.ast
M asterix-app/src/test/resources/runtimets/testsuite.xml
M asterix-app/src/test/resources/runtimets/testsuite_sqlpp.xml
M asterix-om/src/main/java/org/apache/asterix/om/functions/AsterixBuiltinFunctions.java
73 files changed, 1,751 insertions(+), 388 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/29/629/15
-- 
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: 15
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu <buyingyi@gmail.com>
Gerrit-Reviewer: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Jianfeng Jia <jianfeng.jia@gmail.com>
Gerrit-Reviewer: Preston Carman <prestonc@apache.org>
Gerrit-Reviewer: Till Westmann <tillw@apache.org>
Gerrit-Reviewer: Yingyi Bu <buyingyi@gmail.com>

Mime
View raw message