Re: Patch to add a primary key using an existing index
Steve Singer <ssinger@ca.afilias.info>
From: Steve Singer <ssinger@ca.afilias.info>
To: Steve Singer <ssinger_pg@sympatico.ca>
Cc: Gurjeet Singh <singh.gurjeet@gmail.com>, PGSQL Hackers <pgsql-hackers@postgresql.org>
Date: 2010-11-25T20:58:40Z
Lists: pgsql-hackers
Attachments
- replace_pkey_index.revised2.patch.gz (application/x-gzip) patch
On 10-11-22 03:24 PM, Steve Singer wrote: > On 10-11-22 09:37 AM, Gurjeet Singh wrote: >> On Sat, Nov 20, 2010 at 9:00 AM, Steve Singer <ssinger_pg@sympatico.ca > Almost fixed. > I still get an unexpected difference. > > ! DETAIL: cannot create PRIMARY KEY/UNIQUE constraint with a non-unique > index. > CREATE UNIQUE INDEX rpi_idx2 ON rpi_test(a , b); > -- should fail; WITH INDEX option specified more than once. > ALTER TABLE rpi_test ADD PRIMARY KEY (a, b) > --- 35,41 ---- > -- should fail; non-unique > ALTER TABLE rpi_test ADD primary key(a, b) WITH (INDEX = 'rpi_idx1'); > ERROR: "rpi_idx1" is not a unique index > ! DETAIL: Cannot create PRIMARY KEY/UNIQUE constraint using a non-unique > index. The attached version of the patch gets your regression tests to pass. I'm going to mark this as ready for a committer.