]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Fix query jumbling with GROUP BY clauses
authorMichael Paquier <michael@paquier.xyz>
Tue, 13 Jan 2026 23:44:52 +0000 (08:44 +0900)
committerMichael Paquier <michael@paquier.xyz>
Tue, 13 Jan 2026 23:44:52 +0000 (08:44 +0900)
commit9c3caad0264011490134816b2264de7f20b2eb99
treed863cba5e9ad5760e266d59fd9ce4d9dbf488ae8
parent6920fc34531449a5e19f8e81f884f9f1de032b42
Fix query jumbling with GROUP BY clauses

RangeTblEntry.groupexprs was marked with the node attribute
query_jumble_ignore, causing a list of GROUP BY expressions to be
ignored during the query jumbling.  For example, these two queries could
be grouped together within the same query ID:
SELECT count(*) FROM t GROUP BY a;
SELECT count(*) FROM t GROUP BY b;

However, as such queries use different GROUP BY clauses, they should be
split across multiple entries.

This fixes an oversight in 247dea89f761, that has introduced an RTE for
GROUP BY clauses.  Query IDs are documented as being stable across minor
releases, but as this is a regression new to v18 and that we are still
early in its support cycle, a backpatch is exceptionally done as this
has broken a behavior that exists since query jumbling is supported in
core, since its introduction in pg_stat_statements.

The tests of pg_stat_statements are expanded to cover this area, with
patterns involving GROUP BY and GROUPING clauses.

Author: Jian He <jian.universality@gmail.com>
Discussion: https://postgr.es/m/CACJufxEy2W+tCqC7XuJ94r3ivWsM=onKJp94kRFx3hoARjBeFQ@mail.gmail.com
Backpatch-through: 18
contrib/pg_stat_statements/expected/select.out
contrib/pg_stat_statements/sql/select.sql
src/include/nodes/parsenodes.h