RE: DROP CONSTRAINT (UNIQUE) preliminary support

From: "Christopher Kings-Lynne" <chriskl(at)familyhealth(dot)com(dot)au>
To: "Bruce Momjian" <pgman(at)candle(dot)pha(dot)pa(dot)us>
Cc: "Patches" <pgsql-patches(at)postgresql(dot)org>
Subject: RE: DROP CONSTRAINT (UNIQUE) preliminary support
Date: 2001-08-25 03:08:12
Message-ID: ECEHIKNFIMMECLEBJFIGGEKHCBAA.chriskl@familyhealth.com.au
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

Forget this patch entirely - it was just a preview so I could get comment.
But then I got bored and decided to implement ADD UNIQUE and ADD PRIMARY
before worrying about the DROP code.

Discard it - I'll resubmit a proper patch some other time.

Chris

> -----Original Message-----
> From: Bruce Momjian [mailto:pgman(at)candle(dot)pha(dot)pa(dot)us]
> Sent: Saturday, 25 August 2001 1:35 AM
> To: Christopher Kings-Lynne
> Cc: Patches
> Subject: Re: [PATCHES] DROP CONSTRAINT (UNIQUE) preliminary support
>
>
>
> Where did we leave this?
>
> > Hi all,
> >
> > This patch allows ALTER TABLE/DROP CONSTRAINT to drop UNIQUE
> constraints by
> > dropping the relevant index.
> >
> > This is a prelim patch that I need comment on (The @@
> comments), and I've
> > edited some things out by hand, so I don't recommend applying
> it to anything
> > just yet.
> >
> > I have added an IsIndex function to indexcmds.c and a
> RemoveUniqueConstraint
> > to heap.c, plus modified command.c's AlterTableDropConstraint
> function to
> > call RemoveUniqueConstraint.
> >
> > Note that I have disabled inheritance support for unique
> constraints - since
> > it's impossible to have two indices of the same name anyway...
> >
> > Comments please.
> >
> > Chris
>
> [ Attachment, skipping... ]
>
> >
> > ---------------------------(end of broadcast)---------------------------
> > TIP 2: you can get off all lists at once with the unregister command
> > (send "unregister YourEmailAddressHere" to majordomo(at)postgresql(dot)org)
>
> --
> Bruce Momjian | http://candle.pha.pa.us
> pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
> + If your life is a hard drive, | 830 Blythe Avenue
> + Christ can be your backup. | Drexel Hill, Pennsylvania 19026
>

In response to

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2001-08-25 04:47:52 Re: Re: [PATCHES] Patch to include PAM support...
Previous Message Dominic J. Eidson 2001-08-25 01:46:07 Re: [PATCHES] Patch to include PAM support...