pgsql: Fix incorrect handling of subquery pullup in the presence of gro

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Fix incorrect handling of subquery pullup in the presence of gro
Date: 2018-01-12 17:25:00
Message-ID: E1ea34e-0001Gn-Pm@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Fix incorrect handling of subquery pullup in the presence of grouping sets.

If we flatten a subquery whose target list contains constants or
expressions, when those output columns are used in GROUPING SET columns,
the planner was capable of doing the wrong thing by merging a pulled-up
expression into the surrounding expression during const-simplification.
Then the late processing that attempts to match subexpressions to grouping
sets would fail to match those subexpressions to grouping sets, with the
effect that they'd not go to null when expected.

To fix, wrap such subquery outputs in PlaceHolderVars, ensuring that
they preserve their separate identity throughout the planner's expression
processing. This is a bit of a band-aid, because the wrapper defeats
const-simplification even in places where it would be safe to allow.
But a nicer fix would likely be too invasive to back-patch, and the
consequences of the missed optimizations probably aren't large in most
cases.

Back-patch to 9.5 where grouping sets were introduced.

Heikki Linnakangas, with small mods and better test cases by me;
additional review by Andrew Gierth

Discussion: https://postgr.es/m/7dbdcf5c-b5a6-ef89-4958-da212fe10176@iki.fi

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/90947674fc984f5639e3b1bf013435a023aa713b

Modified Files
--------------
src/backend/optimizer/prep/prepjointree.c | 48 +++++++++++++++++++++++++-----
src/test/regress/expected/groupingsets.out | 45 ++++++++++++++++++++++++++++
src/test/regress/sql/groupingsets.sql | 20 +++++++++++++
3 files changed, 105 insertions(+), 8 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2018-01-12 20:47:15 pgsql: Avoid unnecessary failure in SELECT concurrent with ALTER NO INH
Previous Message Michael Meskes 2018-01-12 15:00:49 pgsql: Fix parsing of compatibility mode argument.