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:23 +0000 (20:18 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 24 Apr 2008 20:18:23 +0000 (20:18 +0000)
commit42034630be35f103c903e6b591a8977d9c1afeb7
tree9af0fe82a9e0b29cecd52fc510ecfe6ce34e5c5b
parent90654bee08066a37b0981928948b8f7223a9b623
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