Re: Foreign key validation failure in 18beta1

From: Amul Sul <sulamul(at)gmail(dot)com>
To: jian he <jian(dot)universality(at)gmail(dot)com>
Cc: Tender Wang <tndrwang(at)gmail(dot)com>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, pgsql-hackers(at)lists(dot)postgresql(dot)org, Antonin Houska <ah(at)cybertec(dot)at>
Subject: Re: Foreign key validation failure in 18beta1
Date: 2025-06-02 03:26:27
Message-ID: CAAJ_b960ek4qb-oci1EZ14L+XXw1yihb=JsM3N4eq7DR4QdrrA@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sun, Jun 1, 2025 at 6:05 PM jian he <jian(dot)universality(at)gmail(dot)com> wrote:
>
> On Fri, May 30, 2025 at 6:32 PM Amul Sul <sulamul(at)gmail(dot)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 Content-Type Size
v5-0001-Skip-adding-action-based-foreign-key-constraints-.patch application/x-patch 11.4 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2025-06-02 03:33:23 Re: pgsql: postgres_fdw: Inherit the local transaction's access/deferrable
Previous Message jian he 2025-06-02 03:15:19 Re: CREATE DOMAIN create two not null constraints