Re: pgsql: Invalidate RI fast-path metadata on operator family changes

From: Amit Langote <amitlangote09(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Richard Guo <guofenglinux(at)gmail(dot)com>, Alexander Lakhin <exclusion(at)gmail(dot)com>, pgsql-committers(at)lists(dot)postgresql(dot)org, pgsql-release(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Invalidate RI fast-path metadata on operator family changes
Date: 2026-09-20 04:10:08
Message-ID: CA+HiwqG7dAM2+Jv6RJq6+0L2yS=W2x9A8AV9kNDP2Pa7DCk17w@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers

On Sun, Sep 20, 2026 at 12:34 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Amit Langote <amitlangote09(at)gmail(dot)com> writes:
> > I'm thinking of pushing this to master first and see if it helps. I
> > note in my proposed commit message that back-patching to 19 is
> > deferred until beta4 freeze is over, but maybe I should not wait until
> > then?
>
> Formally, we're in release freeze on v19, so you should get the
> concurrence of pgsql-release@ before pushing something into the
> v19 branch this weekend. It's going to be hard to get meaningful
> input though (seeing that it's late Saturday night USA time, and
> very early Sunday morning European time). Moreover, the longer
> you wait, the fewer buildfarm runs will happen before release wrap;
> and it being a weekend there's not going to be a lot of commit
> activity to help runs happen. So there's no way around some risk
> here.
>
> Having said that, the failure rate seems high enough to be
> annoying (I see six BF failures across master and v19 in the
> 24-ish hours since this went in), and the patch looks pretty
> safe. So I think my vote is to push. I'd counsel asking
> pgsql-release@ as a matter of formality, and pushing if no
> objections arrive within a couple of hours.

Thanks Tom.

Added pgsql-release. For context: the change is regression-test-only.
It wraps the operator-family test added by c62b330912e in a
rolled-back transaction so the family is never visible to concurrently
running tests, which has been causing intermittent plan-shape failures
in window.sql and equivclass.sql on several buildfarm animals (six in
the last ~24h as Tom noted).

Patch I would like to push is attached. I'd like concurrence to push
it to REL_19_STABLE during the freeze.

I'll push to both branches around 15:30 JST (06:30 UTC) unless I hear
objections.

Sorry about the weekend rush.

--
Thanks, Amit Langote

Attachment Content-Type Size
0001-Attempt-to-fix-test-interference-from-foreign_key-op.patch application/octet-stream 5.4 KB

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2026-09-20 04:20:02 Re: pgsql: Invalidate RI fast-path metadata on operator family changes
Previous Message Tom Lane 2026-09-20 03:34:52 Re: pgsql: Invalidate RI fast-path metadata on operator family changes