pgsql: Reject ADD CONSTRAINT NOT NULL if name mismatches existing const - Mailing list pgsql-committers

From Álvaro Herrera
Subject pgsql: Reject ADD CONSTRAINT NOT NULL if name mismatches existing const
Date
Msg-id E1vnEks-000u3v-1B@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Reject ADD CONSTRAINT NOT NULL if name mismatches existing constraint

When using ALTER TABLE ... ADD CONSTRAINT to add a not-null constraint
with an explicit name, we have to ensure that if the column is already
marked NOT NULL, the provided name matches the existing constraint name.
Failing to do so could lead to confusion regarding which constraint
object actually enforces the rule.

This patch adds a check to throw an error if the user tries to add a
named not-null constraint to a column that already has one with a
different name.

Reported-by: yanliang lei <msdnchina@163.com>
Co-authored-by: Álvaro Herrera <alvherre@kurilemu.de>
Co-authored-bu: Srinath Reddy Sadipiralla <srinath2133@gmail.com>
Backpatch-through: 18
Discussion: https://postgr.es/m/19351-8f1c523ead498545%40postgresql.org

Branch
------
REL_18_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/492a69e1407029f8c673484f44aa719a63323d77

Modified Files
--------------
src/backend/catalog/heap.c                |  1 +
src/backend/catalog/pg_constraint.c       | 21 +++++++++++++++++++--
src/include/catalog/pg_constraint.h       |  2 +-
src/test/regress/expected/constraints.out |  6 +++++-
src/test/regress/sql/constraints.sql      |  4 +++-
5 files changed, 29 insertions(+), 5 deletions(-)


pgsql-committers by date:

Previous
From: Peter Eisentraut
Date:
Subject: pgsql: Rename AssertVariableIsOfType to StaticAssertVariableIsOfType
Next
From: Heikki Linnakangas
Date:
Subject: pgsql: Don't hint that you can reconnect when the database is dropped