public inbox for [email protected]  

help / color / mirror / Atom feed

From: Bruce Momjian <[email protected]>
To: Fabien COELHO <[email protected]>
Cc: Tom Lane <[email protected]>
Cc: Jan Wieck <[email protected]>
Cc: PostgreSQL Patches <[email protected]>
Subject: Re: "ALSO" keyword to "CREATE RULE" patch
Date: Thu, 4 Mar 2004 10:36:22 -0500 (EST)
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>

Fabien COELHO wrote:
> 
> > I thought the syntax came from Berkeley.  We can add ALSO if folks like
> > it.  I can't think of cases where we have keywords for both on and off
> > behavior, and allow a default if the keyword is missing.
> 
> ALTER TABLE ... DROP CONSTRAINT ... [ RESTRICT | CASCADE ] ;
> 
> CREATE TABLE ... [ WITH OIDS | WITHOUT OIDS ] ... ;
> 
> CREATE USER ... [ CREATEDB | NOCREATEDB ] ... ;
> 
> IMHO, from the language design point of view, it seems better if all
> options have a name.

Makes sense to me now.  No one liked the non-INSTEAD rule description,
for sure.

Shoot me the patch again and I will put in the the queue.  Thanks.

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  [email protected]               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073

view thread (15+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected], [email protected]
  Subject: Re: "ALSO" keyword to "CREATE RULE" patch
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox