Skip site navigation (1)
Skip section navigation (2)
Peripheral Links
Header And Logo
|
Site Navigation
Re: ADD COLUMN with PRIMARY KEY bug (was: I think this is a BUG?)
- From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
- To: "Brendan Jurd" <direvus(at)gmail(dot)com>
- Cc: pgsql-patches(at)postgresql(dot)org, "Richard Huxton" <dev(at)archonet(dot)com>, kaloyan(at)digsys(dot)bg
- Subject: Re: ADD COLUMN with PRIMARY KEY bug (was: I think this is a BUG?)
- Date: Thu, 24 Apr 2008 15:13:42 -0400
- Message-id: <2574(dot)1209064422(at)sss(dot)pgh(dot)pa(dot)us>
"Brendan Jurd" <direvus(at)gmail(dot)com> writes: > I realised that there's no reason for preparing a separate SetNotNull > subcommand anymore, now that ATExecAddColumn takes care of enforcing > the constraint, so I removed this special case. This part seems to me to be code beautification, not a bug-fix, and shouldn't be back-patched (particularly in view of the fact that we haven't tested the change all that much --- there might be other places depending on the old behavior). Will take care of this. regards, tom lane
- References:
- ADD COLUMN with PRIMARY KEY bug (was: I think this is a BUG?)
- From: Brendan Jurd
- ADD COLUMN with PRIMARY KEY bug (was: I think this is a BUG?)
- Prev by Date: ADD COLUMN with PRIMARY KEY bug (was: I think this is a BUG?)
- Next by Date: Re: ADD COLUMN with PRIMARY KEY bug (was: I think this is a BUG?)
- Previous by thread: ADD COLUMN with PRIMARY KEY bug (was: I think this is a BUG?)
- Next by thread: Re: ADD COLUMN with PRIMARY KEY bug (was: I think this is a BUG?)
- Indexes: