| 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 |
| 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 |