>From Ali Alsuliman <ali.al.solai...@gmail.com>:

Attention is currently required from: Peeyush Gupta, Ali Alsuliman, 
preetha...@apache.org.
Hello Peeyush Gupta, Jenkins, Anon. E. Moose #1000171, preetha...@apache.org,

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

    https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19703

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

Change subject: Revert "[ASTERIXDB-3573][COMP] Do not pull predicates out of 
join condition"
......................................................................

Revert "[ASTERIXDB-3573][COMP] Do not pull predicates out of join condition"

This reverts commit a08dd5bfce1bd82c49d2f1240e94484825de862c.

Reason for revert: performance regression in some queries with complex join 
condition (e.g. CH2 Q19).

Ext-ref: MB-65642

Change-Id: I9036a302d30c65f4596ff2a6d2ea6d16a8af3921
---
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/dynamic-prefixes/avro/one-field/one-field.231.plan
R 
asterixdb/asterix-app/src/test/resources/optimizerts/queries_sqlpp/pull_select_above_eq_join.sqlpp
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/temporal/interval_joins/interval_started_by/interval_started_by.3.query.plan
M asterixdb/asterix-app/src/test/resources/optimizerts/results/ch2/ch2_q17.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/temporal/interval_joins/interval_ended_by/interval_ended_by.3.query.plan
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/dynamic-prefixes/one-field/one-field.231.plan
D 
asterixdb/asterix-app/src/test/resources/optimizerts/results/keep_pred_in_join.plan
M 
hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/PullSelectOutOfEqJoin.java
A 
asterixdb/asterix-app/src/test/resources/optimizerts/results/pull_select_above_eq_join.plan
R 
asterixdb/asterix-app/src/test/resources/optimizerts/results_parser_sqlpp/pull_select_above_eq_join.ast
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/column-filter/ASTERIXDB-3595.001.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/temporal/interval_joins/interval_ends/interval_ends.3.query.plan
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_column/tpch/query-plans/query-plans.08.plan
M asterixdb/asterix-app/src/test/resources/optimizerts/results/ch2/ch2_q19.plan
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results/temporal/interval_joins/interval_starts/interval_starts.3.query.plan
R 
asterixdb/asterix-app/src/test/resources/optimizerts/queries/pull_select_above_eq_join.sqlpp
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/tpch/query-plans/query-plans.08.plan
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/RuleCollections.java
M 
asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/ch2/ch2_q7.plan
19 files changed, 821 insertions(+), 785 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/03/19703/3
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19703
To unsubscribe, or for help writing mail filters, visit 
https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: ionic
Gerrit-Change-Id: I9036a302d30c65f4596ff2a6d2ea6d16a8af3921
Gerrit-Change-Number: 19703
Gerrit-PatchSet: 3
Gerrit-Owner: Ali Alsuliman <ali.al.solai...@gmail.com>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Peeyush Gupta <peeyush.gu...@couchbase.com>
Gerrit-Reviewer: preetha...@apache.org
Gerrit-Attention: Peeyush Gupta <peeyush.gu...@couchbase.com>
Gerrit-Attention: Ali Alsuliman <ali.al.solai...@gmail.com>
Gerrit-Attention: preetha...@apache.org
Gerrit-MessageType: newpatchset

Reply via email to