You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@asterixdb.apache.org by "Yingyi Bu (Code Review)" <do...@asterixdb.incubator.apache.org> on 2016/02/21 22:02:16 UTC
Change in asterixdb[master]: ASTERIXDB-1205: fix union queries.
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 (#16).
Change subject: ASTERIXDB-1205: fix union queries.
......................................................................
ASTERIXDB-1205: fix union queries.
Fixed the AqlExpressionToPlanTranslator for union expression;
Eliminate shared operator references in translated logical plans;
Removed IntroduceUnionRule;
Fixed tuple source for SQL++ join clause;
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/16
--
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: 16
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Yingyi Bu <bu...@gmail.com>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Jianfeng Jia <ji...@gmail.com>
Gerrit-Reviewer: Preston Carman <pr...@apache.org>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Yingyi Bu <bu...@gmail.com>
Gerrit-Reviewer: abdullah alamoudi <ba...@gmail.com>