public inbox for [email protected]  

help / color / mirror / Atom feed

From: Fabien COELHO <[email protected]>
To: Bruce Momjian <[email protected]>
Cc: PostgreSQL Patches <[email protected]>
Subject: Re: "ALSO" keyword to "CREATE RULE" patch
Date: Thu, 4 Mar 2004 09:40:04 +0100 (CET)
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>


Dear Bruce,

> Is ALSO part of the SQL standard?  I can't imagine it is because there
> is no rule mention.

As RULE is not in the SQL standard, ALSO is sure no part of it.

> I guess if we were coding from scratch, we could make the syntax INSTEAD
> or ALSO, but at this point, I don't see adding ALSO as a useful change.

It depends on what you mean by "useful". It is certainly non essential.
I just think it is "user-friendly".

It took me some time to understand that rules were appended by default.
With the "ALSO" keyword it would have been instantaneous.

When I first looked at the syntax with the "INSTEAD" which is optionnal
and without alternative, I thought that it was one of those useless
keywords that are allowed so as to make SQL more English-like, as the
"COLUMN" keyword in the ALTER TABLE syntax. You may argue that I did
not read the documentation in depth, but I'm not sure I'm the only one;-)

The change is very small (2 lines added, 2 lines changed), and upward
compatible, provided that the user has not used ALSO as an object name,
what seems rather unlikely IMVVHO.

Most of the patch deals with the documentation, which is rather ugly
because it keeps telling about "INSTEAD" vs "non-INSTEAD" rules, as there
is no name for the default behavior. I think "ALSO" fixes this issue as it
clarifies the explanations.

Have a nice day,

-- 
Fabien.

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]
  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