Proposed patch for rules with RETURNING

Started by Tom Lanealmost 20 years ago4 messagespatches
Jump to latest
#1Tom Lane
tgl@sss.pgh.pa.us
Attached is a draft patch that makes rules support RETURNING as per my
proposal of earlier today, ie, have the rewriter automatically adjust a
RETURNING clause present in an unconditional INSTEAD rule.  The core of
the patch is barely twenty lines (the code added to rewriteRuleAction)
--- the rest is error checking to reject invalid rules.

Comments, objections?

regards, tom lane

#2Jonah H. Harris
jonah.harris@gmail.com
In reply to: Tom Lane (#1)
Re: Proposed patch for rules with RETURNING

On 9/1/06, Tom Lane <tgl@sss.pgh.pa.us> wrote:

Attached is a draft patch that makes rules support RETURNING ...

What format is that patch in? Either something's wrong or Gmail just
went really wacko.

--
Jonah H. Harris, Software Architect | phone: 732.331.1300
EnterpriseDB Corporation | fax: 732.331.1301
33 Wood Ave S, 2nd Floor | jharris@enterprisedb.com
Iselin, New Jersey 08830 | http://www.enterprisedb.com/

#3Bruce Momjian
bruce@momjian.us
In reply to: Jonah H. Harris (#2)
Re: Proposed patch for rules with RETURNING

Jonah H. Harris wrote:

On 9/1/06, Tom Lane <tgl@sss.pgh.pa.us> wrote:

Attached is a draft patch that makes rules support RETURNING ...

What format is that patch in? Either something's wrong or Gmail just
went really wacko.

I see the attachment fine.

--
Bruce Momjian bruce@momjian.us
EnterpriseDB http://www.enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +

#4Tom Lane
tgl@sss.pgh.pa.us
In reply to: Jonah H. Harris (#2)
Re: Proposed patch for rules with RETURNING

"Jonah H. Harris" <jonah.harris@gmail.com> writes:

On 9/1/06, Tom Lane <tgl@sss.pgh.pa.us> wrote:

Attached is a draft patch that makes rules support RETURNING ...

What format is that patch in? Either something's wrong or Gmail just
went really wacko.

It's just a gzip'd patch diff.

regards, tom lane