From: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
---|---|
To: | jian he <jian(dot)universality(at)gmail(dot)com> |
Cc: | Rushabh Lathia <rushabh(dot)lathia(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Support NOT VALID / VALIDATE constraint options for named NOT NULL constraints |
Date: | 2025-03-20 11:07:29 |
Message-ID: | 202503201107.67xlajjijvkw@alvherre.pgsql |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2025-Mar-20, jian he wrote:
> as you can see the output of `\d+ notnull_tbl1`
> That means the pg_attribute.attnotnull definition is changed.
That's correct, it changed in that way. I propose for the new docs:
> <row>
> <entry role="catalog_table_entry"><para role="column_definition">
> <structfield>attnotnull</structfield> <type>bool</type>
> </para>
> <para>
> This column has a not-null constraint.
> </para></entry>
> </row>
"This column has a possibly unvalidated not-null constraint". The
description for the new column would say "the not-null constraint for
this column is validated" (or the opposite). My recommendation is to
rename the other column from attinvalidnotnull (Rushabh's proposal) to
"attnotnullvalid" and invert the sense of the boolean. I think that's
less confusing.
Thanks
--
Γlvaro Herrera Breisgau, Deutschland β https://www.EnterpriseDB.com/
"If you want to have good ideas, you must have many ideas. Most of them
will be wrong, and what you have to learn is which ones to throw away."
(Linus Pauling)
From | Date | Subject | |
---|---|---|---|
Next Message | Ashutosh Bapat | 2025-03-20 11:23:14 | Re: Enhance 'pg_createsubscriber' to retrieve databases automatically when no database is provided. |
Previous Message | Heikki Linnakangas | 2025-03-20 10:58:14 | Re: Adding support for SSLKEYLOGFILE in the frontend |