Fix ALTER TABLE ADD COLUMN ... PRIMARY KEY so that the new column is correctly
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 24 Apr 2008 20:18:15 +0000 (20:18 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 24 Apr 2008 20:18:15 +0000 (20:18 +0000)
commitdbbceabb5673c7e4c6b785750562920a06283416
tree010e22ecc2d9a2cb3b82f28eefaa9b0d70008182
parentd3fb7ddf56580a451b05745224f0edc75268b659
Fix ALTER TABLE ADD COLUMN ... PRIMARY KEY so that the new column is correctly
checked to see if it's been initialized to all non-nulls.  The implicit NOT
NULL constraint was not being checked during the ALTER (in fact, not even if
there was an explicit NOT NULL too), because ATExecAddColumn neglected to
set the flag needed to make the test happen.  This has been broken since
the capability was first added, in 8.0.

Brendan Jurd, per a report from Kaloyan Iliev.
src/backend/commands/tablecmds.c
src/test/regress/expected/alter_table.out
src/test/regress/sql/alter_table.sql