Changeset: 0c51fd966865 for MonetDB URL: https://dev.monetdb.org/hg/MonetDB/rev/0c51fd966865 Modified Files: sql/backends/monet5/rel_physical.c Branch: default Log Message:
more order-based aggrs could use pre sorting step diffs (22 lines): diff --git a/sql/backends/monet5/rel_physical.c b/sql/backends/monet5/rel_physical.c --- a/sql/backends/monet5/rel_physical.c +++ b/sql/backends/monet5/rel_physical.c @@ -13,6 +13,9 @@ #include "rel_exp.h" #include "rel_rel.h" +#define IS_ORDER_BASED_AGGR(name) (strcmp((name), "quantile") == 0 || strcmp((name), "quantile_avg") == 0 || \ + strcmp((name), "median") == 0 || strcmp((name), "median_avg") == 0) + static sql_rel * rel_add_orderby(visitor *v, sql_rel *rel) { @@ -26,7 +29,7 @@ rel_add_orderby(visitor *v, sql_rel *rel list *aa = e->l; /* for now we only handle one sort order */ - if (strcmp(af->func->base.name, "quantile") == 0 && aa && list_length(aa) == 2) { + if (IS_ORDER_BASED_AGGR(af->func->base.name) && aa && list_length(aa) == 2) { sql_exp *obe = aa->h->data; if (obe) { sql_rel *l = rel->l = rel_project(v->sql->sa, rel->l, rel_projections(v->sql, rel->l, NULL, 1, 1)); _______________________________________________ checkin-list mailing list -- checkin-list@monetdb.org To unsubscribe send an email to checkin-list-le...@monetdb.org