pgsql: Fix CLUSTER tuplesorts on abbreviated expressions. - Mailing list pgsql-committers

From Peter Geoghegan
Subject pgsql: Fix CLUSTER tuplesorts on abbreviated expressions.
Date
Msg-id E1nhKWH-000LU1-OZ@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix CLUSTER tuplesorts on abbreviated expressions.

CLUSTER sort won't use the datum1 SortTuple field when clustering
against an index whose leading key is an expression.  This makes it
unsafe to use the abbreviated keys optimization, which was missed by the
logic that sets up SortSupport state.  Affected tuplesorts output tuples
in a completely bogus order as a result (the wrong SortSupport based
comparator was used for the leading attribute).

This issue is similar to the bug fixed on the master branch by recent
commit cc58eecc5d.  But it's a far older issue, that dates back to the
introduction of the abbreviated keys optimization by commit 4ea51cdfe8.

Backpatch to all supported versions.

Author: Peter Geoghegan <pg@bowt.ie>
Author: Thomas Munro <thomas.munro@gmail.com>
Discussion: https://postgr.es/m/CA+hUKG+bA+bmwD36_oDxAoLrCwZjVtST2fqe=b4=qZcmU7u89A@mail.gmail.com
Backpatch: 10-

Branch
------
REL_12_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/5487585e376dd25d2cabc8a63a33c0286b3a199c

Modified Files
--------------
src/backend/utils/sort/tuplesort.c    |  4 +++-
src/test/regress/expected/cluster.out | 14 ++++++++++++++
src/test/regress/sql/cluster.sql      |  6 ++++++
3 files changed, 23 insertions(+), 1 deletion(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Disallow infinite endpoints in generate_series() for timestamps.
Next
From: Peter Geoghegan
Date:
Subject: pgsql: vacuumlazy.c: MultiXactIds are MXIDs, not XMIDs.