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