Re: doc pg_constraint.convalidated column description need update

From: Robert Treat <rob(at)xzilla(dot)net>
To: Peter Eisentraut <peter(at)eisentraut(dot)org>
Cc: jian he <jian(dot)universality(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: doc pg_constraint.convalidated column description need update
Date: 2025-06-04 20:47:20
Message-ID: CAJSLCQ1xjfT_SNvJtE6Yjh6kZQR=0Ps7veQg4sEuFaqrVJyDdQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Jun 4, 2025 at 9:37 AM Peter Eisentraut <peter(at)eisentraut(dot)org> wrote:
> On 03.05.25 17:42, Robert Treat wrote:
> > I actually think this mix of VALID / ENFORCED is a little convoluted
> > for end users, but I don't think the system catalog documentation is
> > the best place to work that out, and I see we've also not added this
> > type of extra information to other columns in the table that have
> > similar restrictions, so I think removing it is the right move.
> >
> > As such, attached patch removes the above, and attempts some clean up
> > of the documentation in ALTER TABLE to better clarify the behavior
> > around valid/not valid, enforced/not enforced, and how it affects
> > different constraints, with some additional literal tag formatting
> > changes.
>
> I committed the catalog.sgml change. I don't agree with some of the
> other changes, such as changing foreign key to >FOREIGN KEY< etc. It
> was hard to tell if there were any other changes in there that could be
> considered separately from that.
>

I think some of those changes are worth a second stab, so here is an
updated patch which removes the ancillary tagging and corresponding
line wrappings and focuses just on the wording/grammer improvements;
hopefully it will be easier to make sense of.

Robert Treat
https://xzilla.net

Attachment Content-Type Size
v02-cleanup_constraint_validation.txt text/plain 3.2 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Alexander Korotkov 2025-06-04 20:52:50 Re: postgres_fdw could deparse ArrayCoerceExpr
Previous Message Bruce Momjian 2025-06-04 20:46:52 Re: pg_upgrade: warn about roles with md5 passwords