pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Álvaro Herrera <alvherre@kurilemu.de>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Fix error message for ALTER CONSTRAINT ... NOT VALID
Date: Wed, 02 Jul 2025 15:04:14 +0000
Message-ID: <E1uWz0Q-004MzN-0A@gemulon.postgresql.org> (raw)

Fix error message for ALTER CONSTRAINT ... NOT VALID

Trying to alter a constraint so that it becomes NOT VALID results in an
error that assumes the constraint is a foreign key.  This is potentially
wrong, so give a more generic error message.

While at it, give CREATE CONSTRAINT TRIGGER a better error message as
well.

Co-authored-by: jian he <jian.universality@gmail.com>
Co-authored-by: Fujii Masao <masao.fujii@oss.nttdata.com>
Co-authored-by: Álvaro Herrera <alvherre@kurilemu.de>
Co-authored-by: Amul Sul <sulamul@gmail.com>
Discussion: https://postgr.es/m/CACJufxHSp2puxP=q8ZtUGL1F+heapnzqFBZy5ZNGUjUgwjBqTQ@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/c989affb527d330898e92e6223e2218e702176ed

Modified Files
--------------
src/backend/parser/gram.y                 | 6 ++++++
src/test/regress/expected/constraints.out | 5 +++++
src/test/regress/expected/foreign_key.out | 2 +-
src/test/regress/sql/constraints.sql      | 3 +++
4 files changed, 15 insertions(+), 1 deletion(-)



view thread (2+ messages)  latest in thread

Message-ID: <E1uWz0Q-004MzN-0A@gemulon.postgresql.org>
Permalink:  ../E1uWz0Q-004MzN-0A@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1uWz0Q-004MzN-0A@gemulon.postgresql.org

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: alvherre@kurilemu.de, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Fix error message for ALTER CONSTRAINT ... NOT VALID
  In-Reply-To: <E1uWz0Q-004MzN-0A@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox