>From Preetham Poluparthi <[email protected]>:
Attention is currently required from: Ali Alsuliman, Preetham Poluparthi.
Hello Ali Alsuliman, Anon. E. Moose #1000171, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21196?usp=email
to look at the new patch set (#17).
The following approvals got outdated and were removed:
Contrib-2 by Anon. E. Moose #1000171, Integration-Tests-1 by Jenkins,
Verified-1 by Jenkins
Change subject: [ASTERIXDB-3713][COMP] Improve replicate operator placement for
common subplan extraction
......................................................................
[ASTERIXDB-3713][COMP] Improve replicate operator placement for common subplan
extraction
- user model changes: no
- storage format changes: no
- interface changes: no
Details : Adds logic to insert a replicate operator at the divergence point
when bottom-up growth of equivalence classes can no longer find isomorphic
parents. Equivalence classes are now pruned early when it is impossible to add
a replicate (e.g., trivial empty-tuple-source candidates or union-all sinks).
Ext-ref: MB-70862
Change-Id: I686a87ac763c53339f2418b9aa9696fc5cc9e2c5
---
M
asterixdb/asterix-app/src/test/resources/optimizerts/results/extract-common-operators/extract-common-operators.01.plan
M
asterixdb/asterix-app/src/test/resources/optimizerts/results/query_issue3316.plan
M
asterixdb/asterix-app/src/test/resources/optimizerts/results/tpcds/query-ASTERIXDB-1581-correlated-2.plan
M
asterixdb/asterix-app/src/test/resources/optimizerts/results/tpcds/query-ASTERIXDB-1581-correlated.plan
M
asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/extract-common-operators/extract-common-operators.01.plan
M
asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/query_issue3316.plan
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/replicate-operator/union-all/union-all.1.ddl.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/replicate-operator/union-all/union-all.2.update.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/replicate-operator/union-all/union-all.3.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/replicate-operator/union-all/union-all.4.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/replicate-operator/union-sink/union-sink.1.ddl.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/replicate-operator/union-sink/union-sink.2.update.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/replicate-operator/union-sink/union-sink.3.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/replicate-operator/union-sink/union-sink.4.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/results/replicate-operator/union-all/union-all.3.adm
A
asterixdb/asterix-app/src/test/resources/runtimets/results/replicate-operator/union-all/union-all.4.plan
A
asterixdb/asterix-app/src/test/resources/runtimets/results/replicate-operator/union-sink/union-sink.3.adm
A
asterixdb/asterix-app/src/test/resources/runtimets/results/replicate-operator/union-sink/union-sink.4.plan
M
asterixdb/asterix-app/src/test/resources/runtimets/results/tpch/query-plans/query-plans.04.plan
A
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/replicate-operator/union-all/union-all.4.plan
A
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/replicate-operator/union-sink/union-sink.4.plan
M
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/tpch/query-plans/query-plans.04.plan
A
asterixdb/asterix-app/src/test/resources/runtimets/results_column/replicate-operator/union-all/union-all.4.plan
A
asterixdb/asterix-app/src/test/resources/runtimets/results_column/replicate-operator/union-sink/union-sink.4.plan
M
asterixdb/asterix-app/src/test/resources/runtimets/results_column/tpch/query-plans/query-plans.04.plan
A
asterixdb/asterix-app/src/test/resources/runtimets/results_full_parallelism/replicate-operator/union-sink/union-sink.4.plan
A
asterixdb/asterix-app/src/test/resources/runtimets/results_less_parallelism/replicate-operator/union-sink/union-sink.4.plan
M asterixdb/asterix-app/src/test/resources/runtimets/sqlpp_queries.xml
M
hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/ExtractCommonOperatorsRule.java
29 files changed, 7,895 insertions(+), 1,109 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/96/21196/17
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21196?usp=email
To unsubscribe, or for help writing mail filters, visit
https://asterix-gerrit.ics.uci.edu/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: asterixdb
Gerrit-Branch: lumina
Gerrit-Change-Id: I686a87ac763c53339f2418b9aa9696fc5cc9e2c5
Gerrit-Change-Number: 21196
Gerrit-PatchSet: 17
Gerrit-Owner: Preetham Poluparthi <[email protected]>
Gerrit-Reviewer: Ali Alsuliman <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Preetham Poluparthi <[email protected]>
Gerrit-Attention: Ali Alsuliman <[email protected]>
Gerrit-Attention: Preetham Poluparthi <[email protected]>
