Re: addFkRecurseReferencing use unassigned fkconstraint->fk_with_period value

From: solai v <solai(dot)cdac(at)gmail(dot)com>
To: Paul A Jungwirth <pj(at)illuminatedcomputing(dot)com>
Cc: Haibo Yan <tristan(dot)yim(at)gmail(dot)com>, Andrew Krylosov <krylosov(dot)andrew(at)gmail(dot)com>, jian he <jian(dot)universality(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: addFkRecurseReferencing use unassigned fkconstraint->fk_with_period value
Date: 2026-10-14 07:23:44
Message-ID: CAF0whucG7kWmAW21O8AfzxqxxOftgmhN+L4uTydSYynAjy1Miw@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

I tested the v3 patch for the PERIOD foreign-key validation issue.
Before applying the patch, I reproduced the bug: attaching a partition
containing an invalid existing row succeeded unexpectedly, while
inserting the same invalid row afterward failed with a foreign-key
violation.
After applying the patch and rebuilding PostgreSQL, the same partition
attachment was correctly rejected with a foreign-key violation.
I also ran 'make check' and all regression tests passed. The added
regression tests cover 'VALIDATE CONSTRAINT', 'ALTER CONSTRAINT ...
ENFORCED', and 'ATTACH PARTITION', including cases where existing rows
are invalid or their periods are covered after correction.
Based on my testing, the patch fixes the reproduced
partition-attachment issue, and I did not observe regression-test
failures.

Regards,
solai

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message David Rowley 2026-10-14 07:27:53 Re: Reducing relcache memory usage: deduping index shapes
Previous Message Heikki Linnakangas 2026-10-14 07:17:44 Re: pg_upgrade fails with a config-only datadir and no --new-bindir