"ADD" notice in CREATE/ALTER TABLE

Started by Bruce Momjianabout 24 years ago3 messages
#1Bruce Momjian
pgman@candle.pha.pa.us

Christopher wanted "ADD" used in:

ALTER TABLE / ADD PRIMARY-UNIQUE
^^^

but not in:

CREATE TABLE / PRIMARY-UNIQUE

The problem is that the same analyze.c function is used for both cases.
I have added code to conditionally use "ADD" only in fhe first case.

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026
#2Philip Warner
pjw@rhyme.com.au
In reply to: Bruce Momjian (#1)
Re: "ADD" notice in CREATE/ALTER TABLE

At 00:40 4/11/01 -0500, Bruce Momjian wrote:

Christopher wanted "ADD" used in:

ALTER TABLE / ADD PRIMARY-UNIQUE
^^^

but not in:

CREATE TABLE / PRIMARY-UNIQUE

The problem is that the same analyze.c function is used for both cases.
I have added code to conditionally use "ADD" only in fhe first case.

What's this in relation to? Shouldn't the syntax be:

ALTER TABLE ADD CONSTRAINT....

Or am I misreading the original message?

----------------------------------------------------------------
Philip Warner | __---_____
Albatross Consulting Pty. Ltd. |----/ - \
(A.B.N. 75 008 659 498) | /(@) ______---_
Tel: (+61) 0500 83 82 81 | _________ \
Fax: (+61) 0500 83 82 82 | ___________ |
Http://www.rhyme.com.au | / \|
| --________--
PGP key available upon request, | /
and from pgp5.ai.mit.edu:11371 |/

#3Bruce Momjian
pgman@candle.pha.pa.us
In reply to: Philip Warner (#2)
Re: "ADD" notice in CREATE/ALTER TABLE

At 00:40 4/11/01 -0500, Bruce Momjian wrote:

Christopher wanted "ADD" used in:

ALTER TABLE / ADD PRIMARY-UNIQUE
^^^

but not in:

CREATE TABLE / PRIMARY-UNIQUE

The problem is that the same analyze.c function is used for both cases.
I have added code to conditionally use "ADD" only in fhe first case.

What's this in relation to? Shouldn't the syntax be:

ALTER TABLE ADD CONSTRAINT....

Or am I misreading the original message?

Uhh, this was a change if elog(NOTICE) display for Christopher. He
added alter table regression tests. The old message didn't have the
ADD, and I added that. My guess is that we don't spell out the word
CONSTRAINT in the message, just the ALTER and ADD parts.

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026