Changeset: 004d7b12ddb7 for MonetDB URL: https://dev.monetdb.org/hg/MonetDB/rev/004d7b12ddb7 Modified Files: sql/test/rel-optimizers/optimize-proj/Tests/groupby-cse.test Branch: optimizer-testing Log Message:
Adds comments for not implemented test cases diffs (23 lines): diff --git a/sql/test/rel-optimizers/optimize-proj/Tests/groupby-cse.test b/sql/test/rel-optimizers/optimize-proj/Tests/groupby-cse.test --- a/sql/test/rel-optimizers/optimize-proj/Tests/groupby-cse.test +++ b/sql/test/rel-optimizers/optimize-proj/Tests/groupby-cse.test @@ -487,11 +487,16 @@ 2 # the projections ## GROUPBY <--input-- SEMI -# TODO +# this would not make sense since the alias cannot (probably?) exist in +# the expressions of the semijoin operator. and since the groupby_cse is +# only looking in the lower subrel it will not be triggered ## GROUPBY <--input-- UNION -# TODO +# TODO: the union operator introduces a distinct in the form of group +# bys which are NOT processed by the groupby-cse optimizer. this must be +# investigated more eg +# select c1 ca, c1 cb from foo union select c1 ca, c1 cb from bar; ## GROUPBY <--input-- MERGE -# TODO +# not applicable _______________________________________________ checkin-list mailing list -- checkin-list@monetdb.org To unsubscribe send an email to checkin-list-le...@monetdb.org
