Re: [PATCH] Rebuild CHECK constraints after generated column SET EXPRESSION - Mailing list pgsql-hackers

From jian he
Subject Re: [PATCH] Rebuild CHECK constraints after generated column SET EXPRESSION
Date
Msg-id CACJufxFG+vn8vdHAedGuqwb9iC58HHPGSEaLwm6_Tcv-3AyH=w@mail.gmail.com
Whole thread
In response to Re: [PATCH] Rebuild CHECK constraints after generated column SET EXPRESSION  (Zsolt Parragi <zsolt.parragi@percona.com>)
Responses Re: [PATCH] Rebuild CHECK constraints after generated column SET EXPRESSION
Re: [PATCH] Rebuild CHECK constraints after generated column SET EXPRESSION
List pgsql-hackers
On Fri, May 29, 2026 at 6:58 AM Zsolt Parragi <zsolt.parragi@percona.com> wrote:
>
> Looks good, I have one nitpick question:
>
> +                       if (isnull)
> +                               elog(WARNING, "null conbin for relation \"%s\"",
> +                                        RelationGetRelationName(rel));
>
> Shouldn't this be an error instead? Most existing checks treat it as
> an error, and that seems better to me as this shouldn't happen
> normally?
>
OK.

I changed it to:
> +                       if (isnull)
> +                               elog(ERROR, "null conbin for relation \"%s\"",
> +                                        RelationGetRelationName(rel));

I also slightly rephrased the commit message.



--
jian
https://www.enterprisedb.com/

Attachment

pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: [RFC PATCH v2 RESEND 0/10] Umbra: a remap-aware smgr prototype
Next
From: Andrey Chernyy
Date:
Subject: Re: [PATCH] Fix libxml leaks in contrib/xml2 XPath functions