pgsql: Fix CLUSTER on expression indexes. - Mailing list pgsql-committers

From Andres Freund
Subject pgsql: Fix CLUSTER on expression indexes.
Date
Msg-id E1iKQtc-00049G-W8@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix CLUSTER on expression indexes.

Since the introduction of different slot types, in 1a0586de3657, we
create a virtual slot in tuplesort_begin_cluster(). While that looks
right, it unfortunately doesn't actually work, as ExecStoreHeapTuple()
is used to store tuples in the slot. Unfortunately no regression tests
for CLUSTER on expression indexes existed so far.

Fix the slot type, and add bare bones tests for CLUSTER on expression
indexes.

Reported-By: Justin Pryzby
Author: Andres Freund
Discussion: https://postgr.es/m/20191011210320.GS10470@telsasoft.com
Backpatch: 12, like 1a0586de3657

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/cef82eda1464193ab84a58610a388572d456c8c5

Modified Files
--------------
src/backend/utils/sort/tuplesort.c    |   2 +-
src/test/regress/expected/cluster.out | 107 ++++++++++++++++++++++++++++++++++
src/test/regress/sql/cluster.sql      |  37 ++++++++++++
3 files changed, 145 insertions(+), 1 deletion(-)


pgsql-committers by date:

Previous
From: Tomas Vondra
Date:
Subject: pgsql: Correct reference to pg_catalog.regtype in pg_upgrade query
Next
From: Andres Freund
Date:
Subject: pgsql: Fix CLUSTER on expression indexes.