pgsql: Fix case of window function + aggregate + GROUP BY expression.
От | Tom Lane |
---|---|
Тема | pgsql: Fix case of window function + aggregate + GROUP BY expression. |
Дата | |
Msg-id | E1TCBPR-0003Y6-NI@gemulon.postgresql.org обсуждение исходный текст |
Список | pgsql-committers |
Fix case of window function + aggregate + GROUP BY expression. In commit 1bc16a946008a7cbb33a9a06a7c6765a807d7f59 I added a minor optimization to drop the component variables of a GROUP BY expression from the target list computed at the aggregation level of a query, if those Vars weren't referenced elsewhere in the tlist. However, I overlooked that the window-function planning code would deconstruct such expressions and thus need to have access to their component variables. Fix it to not do that. While at it, I removed the distinction between volatile and nonvolatile window partition/order expressions: the code now computes all of them at the aggregation level. This saves a relatively expensive check for volatility, and it's unclear that the resulting plan isn't better anyway. Per bug #7535 from Louis-David Mitterrand. Back-patch to 9.2. Branch ------ master Details ------- http://git.postgresql.org/pg/commitdiff/a20993608a65b9896b4c05cb7061bc60d6f1840a Modified Files -------------- src/backend/optimizer/plan/planner.c | 163 +++++++++++++++++++++++++--------- src/test/regress/expected/window.out | 12 +++ src/test/regress/sql/window.sql | 6 ++ 3 files changed, 137 insertions(+), 44 deletions(-)
В списке pgsql-committers по дате отправления: