Re: Foreign key isolation tests

From: Paul A Jungwirth <pj(at)illuminatedcomputing(dot)com>
To: Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com>
Cc: Rustam ALLAKOV <rustamallakov(at)gmail(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: Foreign key isolation tests
Date: 2025-07-18 23:25:54
Message-ID: CA+renyUFCJKY0dkGY9mxBk+SRG63bi1y1piWfnjWpB_+3LLRnQ@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Jul 18, 2025 at 3:29 AM Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com> wrote:
> I agree that additional testing in this area is valuable. Patch 0001
> looks reasonable to me, on a quick read-through. In patch 0002, I
> think it would be valuable to also test updating the parent row to
> time periods consistent and not consistent with the insert, to confirm
> that that behaves correctly.

Thanks for taking a look! Here are new patches with those extra tests.
There are extensive regress tests already, so I just tested the same
concurrency pattern. I think the results are okay. I do get a
can't-serialize exception for a couple valid changes under REPEATABLE
READ and SERIALIZE, but I think they are expected and not a bug. (I
think you would see the same thing outside of FKs.)

--
Paul ~{:-)
pj(at)illuminatedcomputing(dot)com

Attachment Content-Type Size
v3-0001-Fill-testing-gap-for-possible-referential-integri.patch text/x-patch 7.1 KB
v3-0003-Improve-comment-about-snapshot-macros.patch text/x-patch 1.3 KB
v3-0002-Add-test-for-temporal-referential-integrity.patch text/x-patch 15.9 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Jacob Champion 2025-07-18 23:31:21 Re: [PoC] Federated Authn/z with OAUTHBEARER
Previous Message Hannu Krosing 2025-07-18 22:31:03 Re: Support for 8-byte TOAST values (aka the TOAST infinite loop problem)