>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