Re: Foreign key validation failure in 18beta1 - Mailing list pgsql-hackers

From Amul Sul
Subject Re: Foreign key validation failure in 18beta1
Date
Msg-id CAAJ_b960ek4qb-oci1EZ14L+XXw1yihb=JsM3N4eq7DR4QdrrA@mail.gmail.com
Whole thread Raw
In response to Re: Foreign key validation failure in 18beta1  (Tender Wang <tndrwang@gmail.com>)
List pgsql-hackers
On Sun, Jun 1, 2025 at 6:05 PM jian he <jian.universality@gmail.com> wrote:
>
> On Fri, May 30, 2025 at 6:32 PM Amul Sul <sulamul@gmail.com> wrote:
> >
[...]
>
> + * Note that validation should be performed against the referencing
> + * root table only, not its child partitions. See
> + * QueueFKConstraintValidation() for more details.
>   */
>         if (rel->rd_rel->relkind == RELKIND_RELATION &&
>             currcon->confrelid == pkrelid)
>         {
>             AlteredTableInfo *tab;
>             NewConstraint *newcon;
>             newcon = (NewConstraint *) palloc0(sizeof(NewConstraint));
>             ....
>         }
>
> in the comments "referencing" should be "referenced"?
>
Yes, fixed in the attached version.

> other than that, it looks good.
>
Thanks for the review.

Regards,
Amul

Attachment

pgsql-hackers by date:

Previous
From: jian he
Date:
Subject: Re: ALTER TABLE ALTER CONSTRAINT misleading error message
Next
From: Michael Paquier
Date:
Subject: Re: pgsql: postgres_fdw: Inherit the local transaction's access/deferrable