Thread: Proposed patch for rules with RETURNING

Proposed patch for rules with RETURNING

From
Tom Lane
Date:
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

Attachment

Re: Proposed patch for rules with RETURNING

From
"Jonah H. Harris"
Date:
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/

Re: Proposed patch for rules with RETURNING

From
Bruce Momjian
Date:
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. +

Re: Proposed patch for rules with RETURNING

From
Tom Lane
Date:
"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