Re: Add regression test for ALTER TABLE / ADD UNIQUE

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Christopher Kings-Lynne <chriskl(at)familyhealth(dot)com(dot)au>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: Add regression test for ALTER TABLE / ADD UNIQUE
Date: 2001-10-31 05:21:37
Message-ID: 200110310521.f9V5LcZ21154@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

> > I would like to apply the following patch to make your patch show ADD
> > UNIQUE instead of just UNIQUE and add spacing around the slash in a few
> > cases that were inconsistent.
>
> You might want to change the 'PRIMARY KEY' bit to 'ADD PRIMARY KEY' also,
> but then again I haven't written that code yet!
>
> > The only question I have is how to handle backend/po files. I can
> > easily add the spaces to match the new format if that is OK with the po
> > maintainers.
> >
> > I will hold your patch and mine in the queue to see if people have
> > comments on this and apply accordingly.
>
> Cool.

"ADD" added and regression expected output modified to match.

--
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 Serguei Mokhov 2001-11-04 19:55:51 Update: nls-ru-libpq-20011104-1449.tar.gz
Previous Message Bruce Momjian 2001-10-31 05:21:07 Re: Add regression test for ALTER TABLE / ADD UNIQUE