| From: | Peter Eisentraut <peter(at)eisentraut(dot)org> |
|---|---|
| To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
| Subject: | pgsql: Fix assertion failures in DELETE FOR PORTION OF tuple routing |
| Date: | 2026-09-01 19:19:04 |
| Message-ID: | E1x1U0d-000000034o4-38wQ@gemulon.postgresql.org |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers |
Fix assertion failures in DELETE FOR PORTION OF tuple routing
We must handle cases where an inserted temporal leftover gets routed to
a different partition than the original tuple's. This happens if the
partition key depends on the application-time column used in the FOR
PORTION OF clause. Everything works for UPDATE, but for DELETE we can
hit some Asserts, since that operation never needed to be covered
before. One Assert is for the RETURNING list; the other, for WITH CHECK
OPTION. This commit updates both to include CMD_DELETE.
Author: Paul A Jungwirth <pj(at)illuminatedcomputing(dot)com>
Reported-by: Peter Eisentraut <peter(at)eisentraut(dot)org>
Discussion: https://postgr.es/m/03276924-6d55-470d-8192-a49c053c940f@eisentraut.org
Branch
------
REL_19_STABLE
Details
-------
https://git.postgresql.org/pg/commitdiff/938f4f8f343415f5ce17d0e3f0736b2c118d422e
Modified Files
--------------
src/backend/executor/execPartition.c | 22 +++++++---
src/test/regress/expected/for_portion_of.out | 59 +++++++++++++++++++++++++++
src/test/regress/expected/updatable_views.out | 45 ++++++++++++++++++++
src/test/regress/sql/for_portion_of.sql | 42 +++++++++++++++++++
src/test/regress/sql/updatable_views.sql | 33 +++++++++++++++
5 files changed, 195 insertions(+), 6 deletions(-)
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Nathan Bossart | 2026-09-01 19:21:46 | pgsql: Fix spurious errors in COPY FROM (FORMAT {text,csv}). |
| Previous Message | Andres Freund | 2026-09-01 18:50:24 | pgsql: Make after-startup shmem failure test reliable |