Re: Patch to add a primary key using an existing index
Robert Haas <robertmhaas@gmail.com>
From: Robert Haas <robertmhaas@gmail.com>
To: Itagaki Takahiro <itagaki.takahiro@gmail.com>
Cc: Gurjeet Singh <singh.gurjeet@gmail.com>, Steve Singer <ssinger@ca.afilias.info>, Steve Singer <ssinger_pg@sympatico.ca>, PGSQL Hackers <pgsql-hackers@postgresql.org>
Date: 2010-11-29T01:40:08Z
Lists: pgsql-hackers
On Sun, Nov 28, 2010 at 8:06 PM, Itagaki Takahiro <itagaki.takahiro@gmail.com> wrote: > On Fri, Nov 26, 2010 at 05:58, Steve Singer <ssinger@ca.afilias.info> wrote: >> The attached version of the patch gets your regression tests to pass. >> I'm going to mark this as ready for a committer. > > I think we need more discussions about the syntax: > ALTER TABLE table_name ADD PRIMARY KEY (...) WITH (INDEX='index_name') Why not: ALTER TABLE table_name ADD PRIMARY KEY (...) INDEX index_name; -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company