Changeset: caa7dd79c4c2 for MonetDB
URL: https://dev.monetdb.org/hg/MonetDB/rev/caa7dd79c4c2
Modified Files:
sql/backends/monet5/bin_partition_by_slice.c
Branch: pp_hashjoin
Log Message:
only use results from the groupby not from the current output (as we renumber
which would lead to issues later)
Optimizers should cleanup this issue (of multiple column refs in groupby
results).
diffs (14 lines):
diff --git a/sql/backends/monet5/bin_partition_by_slice.c
b/sql/backends/monet5/bin_partition_by_slice.c
--- a/sql/backends/monet5/bin_partition_by_slice.c
+++ b/sql/backends/monet5/bin_partition_by_slice.c
@@ -1088,9 +1088,7 @@ rel2bin_groupby_pp(backend *be, sql_rel
grp = pgrp;
continue;
}
- /* first look in the current aggr list (l) and group by column
list */
- if (l && !aggrstmt && aggrexp->type == e_column)
- aggrstmt = list_find_column_nid(be, l, aggrexp->nid);
+ /* first try to look up in the group by column list */
if (gbexps && !aggrstmt && aggrexp->type == e_column) {
aggrstmt = list_find_column_nid(be, gbexps,
aggrexp->nid);
if ((!be->pipeline || !_2phases) && aggrstmt &&
groupby) {
_______________________________________________
checkin-list mailing list -- [email protected]
To unsubscribe send an email to [email protected]
