pgsql: Forbid marking an identity column as nullable.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Forbid marking an identity column as nullable.
Date: 2021-03-12 16:08:55
Message-ID: E1lKkLP-0004ne-GQ@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Forbid marking an identity column as nullable.

GENERATED ALWAYS AS IDENTITY implies NOT NULL, but the code failed
to complain if you overrode that with "GENERATED ALWAYS AS IDENTITY
NULL". One might think the old behavior was a feature, but it was
inconsistent because the outcome varied depending on the order of
the clauses, so it seems to have been just an oversight.

Per bug #16913 from Pavel Boev. Back-patch to v10 where identity
columns were introduced.

Vik Fearing (minor tweaks by me)

Discussion: https://postgr.es/m/16913-3b5198410f67d8c6@postgresql.org

Branch
------
REL_12_STABLE

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

Modified Files
--------------
doc/src/sgml/ref/create_table.sgml | 1 +
src/backend/parser/parse_utilcmd.c | 10 ++++++++++
src/test/regress/expected/identity.out | 13 +++++++++++++
src/test/regress/sql/identity.sql | 9 +++++++++
4 files changed, 33 insertions(+)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2021-03-12 17:20:35 pgsql: Fix race condition in psql \e's detection of file modification.
Previous Message Thomas Munro 2021-03-12 11:05:24 pgsql: Specialize checkpointer sort functions.