asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Wenhai Li (Code Review)" <>
Subject Change in asterixdb[master]: ASTERIXDB-1487: fix the wrong plan when we prune the selecti...
Date Fri, 23 Sep 2016 07:17:30 GMT
Hello Jenkins,

I'd like you to reexamine a change.  Please visit

to look at the new patch set (#15).

Change subject: ASTERIXDB-1487: fix the wrong plan when we prune the selective branch.

ASTERIXDB-1487: fix the wrong plan when we prune the selective branch.

1. Add the test case of ASTERIX-1487 with single join branch required.
2. Disable the join branch pruning in case of unnestmap following datasourcescan.
   - We need to prune the join branch when it is NOT required by the upstream operators and
its generated join key is derived from the same DATASOURCE of the other branch.
   - We SHOULD NOT prune the join branch if there exists a selective operator (UNNESTMAP,
LOUNNESTMAP, LIMIT, SELECT) located between the join operator and DATASOURCESCAN.

Change-Id: I1aef69a2278853fd9f8020da6639331b367ed5ad
M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/
M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/
A asterixdb/asterix-app/src/test/resources/optimizerts/queries/select-self-join.aql
M asterixdb/asterix-app/src/test/resources/optimizerts/results/inverted-index-join/issue741.plan
A asterixdb/asterix-app/src/test/resources/optimizerts/results/select-self-join.plan
A asterixdb/asterix-app/src/test/resources/runtimets/queries/fuzzyjoin/dblp-csx-aqlplus_5/dblp-csx-aqlplus_5.1.ddl.aql
A asterixdb/asterix-app/src/test/resources/runtimets/queries/fuzzyjoin/dblp-csx-aqlplus_5/dblp-csx-aqlplus_5.2.update.aql
A asterixdb/asterix-app/src/test/resources/runtimets/queries/fuzzyjoin/dblp-csx-aqlplus_5/dblp-csx-aqlplus_5.3.query.aql
A asterixdb/asterix-app/src/test/resources/runtimets/results/fuzzyjoin/dblp-csx-aqlplus_5/dblp-csx-aqlplus_5.1.adm
M asterixdb/asterix-app/src/test/resources/runtimets/testsuite.xml
10 files changed, 225 insertions(+), 21 deletions(-)

  git pull ssh:// refs/changes/19/1119/15
To view, visit
To unsubscribe, visit

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I1aef69a2278853fd9f8020da6639331b367ed5ad
Gerrit-PatchSet: 15
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Wenhai Li <>
Gerrit-Reviewer: Chen Li <>
Gerrit-Reviewer: Jenkins <>
Gerrit-Reviewer: Jianfeng Jia <>
Gerrit-Reviewer: Taewoo Kim <>
Gerrit-Reviewer: Till Westmann <>
Gerrit-Reviewer: Wenhai Li <>
Gerrit-Reviewer: Yingyi Bu <>
Gerrit-Reviewer: Yingyi Bu <>

View raw message