agora inbox for pgsql-committers@postgresql.orghelp / color / mirror / Atom feed
pgsql: Fix assertion failures in DELETE FOR PORTION OF tuple routing 2+ messages / 1 participants [nested] [flat]
* pgsql: Fix assertion failures in DELETE FOR PORTION OF tuple routing @ 2026-09-01 19:19 Peter Eisentraut <peter@eisentraut.org> 0 siblings, 0 replies; 2+ messages in thread From: Peter Eisentraut @ 2026-09-01 19:19 UTC (permalink / raw) To: pgsql-committers@lists.postgresql.org 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@illuminatedcomputing.com> Reported-by: Peter Eisentraut <peter@eisentraut.org> Discussion: https://postgr.es/m/03276924-6d55-470d-8192-a49c053c940f@eisentraut.org Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/610976420a331ab007621ae30f664d17c8d2eebe 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(-) ^ permalink raw reply [nested|flat] 2+ messages in thread
* pgsql: Fix assertion failures in DELETE FOR PORTION OF tuple routing @ 2026-09-01 19:19 Peter Eisentraut <peter@eisentraut.org> 0 siblings, 0 replies; 2+ messages in thread From: Peter Eisentraut @ 2026-09-01 19:19 UTC (permalink / raw) To: pgsql-committers@lists.postgresql.org 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@illuminatedcomputing.com> Reported-by: Peter Eisentraut <peter@eisentraut.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(-) ^ permalink raw reply [nested|flat] 2+ messages in thread
end of thread, other threads:[~2026-09-01 19:19 UTC | newest] Thread overview: 2+ messages (download: mbox mbox.gz follow: Atom feed) -- links below jump to the message on this page -- 2026-09-01 19:19 pgsql: Fix assertion failures in DELETE FOR PORTION OF tuple routing Peter Eisentraut <peter@eisentraut.org> 2026-09-01 19:19 pgsql: Fix assertion failures in DELETE FOR PORTION OF tuple routing Peter Eisentraut <peter@eisentraut.org>
This inbox is served by agora; see mirroring instructions for how to clone and mirror all data and code used for this inbox