Conversation
|
Interestingly, this pass actually discovers unused aggregate functions which are introduced by the Calcite decorrelator! |
71af2a8 to
cba90af
Compare
cba90af to
bcd07b4
Compare
887e62d to
03616ff
Compare
|
Calcite compiles the following query: To a plan containing the following fragment: By using a |
03616ff to
d7551db
Compare
Signed-off-by: Mihai Budiu <mbudiu@feldera.com>
d7551db to
0e3c38e
Compare
|
The analysis can also discover that the following SUM aggregate is not used, since it's used only for ORDER BY, which is ignored by default: |
|
However, if an aggregate is evaluated just for side-effects (i.e., crash on overflow), removing aggregates is not sound. |
Fixes #5541