Re: Foreign key isolation tests

From: Paul A Jungwirth <pj(at)illuminatedcomputing(dot)com>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com>, Rustam ALLAKOV <rustamallakov(at)gmail(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: Foreign key isolation tests
Date: 2025-08-11 20:18:03
Message-ID: CA+renyX+UpmRuDZjG91hz_cQsz=+Ei3kehsf1mtFB99hPD-H5w@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Aug 11, 2025 at 8:03 AM Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
>
> On Fri, Jul 18, 2025 at 7:26 PM Paul A Jungwirth
> <pj(at)illuminatedcomputing(dot)com> wrote:
> > 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.)
>
> 0001 and 0003 look OK to me on a quick read-through. 0002 seems to do
> something horrible to isolation_schedule.

Ugh, sorry about that. Here is a fix.

Yours,

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

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

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2025-08-11 20:19:45 Re: Some ExecSeqScan optimizations
Previous Message Peter Geoghegan 2025-08-11 20:14:31 Re: index prefetching