Changeset: c987411ae9ae for MonetDB
URL: https://dev.monetdb.org/hg/MonetDB/rev/c987411ae9ae
Modified Files:
        sql/server/rel_optimize_proj.c
Branch: pp_hashjoin
Log Message:

keep old classic rewriter/optimizers enabled for rewrites of distinct


diffs (35 lines):

diff --git a/sql/server/rel_optimize_proj.c b/sql/server/rel_optimize_proj.c
--- a/sql/server/rel_optimize_proj.c
+++ b/sql/server/rel_optimize_proj.c
@@ -3347,6 +3347,7 @@ rel_project_select_exp(visitor *v, sql_r
 static sql_rel *
 rel_optimize_projections_(visitor *v, sql_rel *rel)
 {
+       ATOMIC_TYPE oahash_enabled = (1U<<19);
        rel = rel_project_cse(v, rel);
        rel = rel_project_select_exp(v, rel);
        rel = rel_use_equality_exps(v, rel);
@@ -3361,11 +3362,11 @@ rel_optimize_projections_(visitor *v, sq
                rel = rel_simplify_groupby_columns(v, rel);
        }
        rel = rel_groupby_cse(v, rel);
-       if (0) rel = rel_push_aggr_down(v, rel);
+       if (!oahash_enabled) rel = rel_push_aggr_down(v, rel);
        rel = rel_push_groupby_down(v, rel);
        rel = rel_reduce_groupby_exps(v, rel);
        rel = rel_distinct_aggregate_on_unique_values(v, rel);
-       if (0) rel = rel_groupby_distinct(v, rel);
+       if (!oahash_enabled) rel = rel_groupby_distinct(v, rel);
        rel = rel_push_count_down(v, rel);
 
        /* only when value_based_opt is on, ie not for dependency resolution */
@@ -3522,7 +3523,8 @@ rel_merge_unions(visitor *v, sql_rel *re
 static inline sql_rel *
 rel_push_join_down_munion(visitor *v, sql_rel *rel)
 {
-       if (0 && ((is_join(rel->op) && !is_outerjoin(rel->op) && 
!is_single(rel)) || is_semi(rel->op))) {
+       ATOMIC_TYPE oahash_enabled = (1U<<19);
+       if (!oahash_enabled && ((is_join(rel->op) && !is_outerjoin(rel->op) && 
!is_single(rel)) || is_semi(rel->op))) {
                sql_rel *l = rel->l, *r = rel->r, *ol = l, *or = r;
                list *exps = rel->exps, *attr = rel->attr;
                sql_exp *je = NULL;
_______________________________________________
checkin-list mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to