Re: pgsql: Clean up all relid fields of RestrictInfos during join removal. - Mailing list pgsql-committers

From Michael Paquier
Subject Re: pgsql: Clean up all relid fields of RestrictInfos during join removal.
Date
Msg-id aeawnyFl5lwsG2dC@paquier.xyz
Whole thread
In response to pgsql: Clean up all relid fields of RestrictInfos during join removal.  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: pgsql: Clean up all relid fields of RestrictInfos during join removal.
List pgsql-committers
Hi Tom,

On Mon, Apr 20, 2026 at 06:48:35PM +0000, Tom Lane wrote:
> Clean up all relid fields of RestrictInfos during join removal.
>
> The original implementation of remove_rel_from_restrictinfo()
> thought it could skate by with removing no-longer-valid relid
> bits from only the clause_relids and required_relids fields.
> This is quite bogus, although somehow we had not run across a
> counterexample before now.  At minimum, the left_relids and
> right_relids fields need to be fixed because they will be
> examined later by clause_sides_match_join().  But it seems
> pretty foolish not to fix all the relid fields, so do that.

prion looks unhappy on this one:
https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=prion&dt=2026-04-20%2022%3A50%3A01

This uses -DRELCACHE_FORCE_RELEASE -DCATCACHE_FORCE_RELEASE.
--
Michael

Attachment

pgsql-committers by date:

Previous
From: Jeff Davis
Date:
Subject: pgsql: Fix callers of unicode_strtitle() using srclen == -1.
Next
From: Richard Guo
Date:
Subject: Re: pgsql: Clean up all relid fields of RestrictInfos during join removal.