Re: WIP: Triggers on VIEWs - Mailing list pgsql-hackers

From Tom Lane
Subject Re: WIP: Triggers on VIEWs
Date
Msg-id 14009.1286560651@sss.pgh.pa.us
Whole thread Raw
In response to Re: WIP: Triggers on VIEWs  (Robert Haas <robertmhaas@gmail.com>)
List pgsql-hackers
Robert Haas <robertmhaas@gmail.com> writes:
> On Fri, Oct 8, 2010 at 11:50 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>> Although we already have macros TRIGGER_FIRED_AFTER/TRIGGER_FIRED_BEFORE
>> that seem to mask the details here, the changes you had to make in
>> contrib illustrate that the macros' callers could still be embedding this
>> basic mistake of testing "!before" when they mean "after" or vice versa.
>> I wonder whether we should intentionally rename the macros to force
>> people to take another look at their logic. �Or is that going too far?
>> Comments anyone?

> I'm less sold on this one.

I'm not sold on it either, just wanted to run it up the flagpole to see
if anyone would salute.  For the moment I'm thinking that calling out
the point in the 9.1 release notes should be sufficient.  I made an
extra commit to make sure the issue is salient in the commit log.
        regards, tom lane


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: GIN vs. Partial Indexes
Next
From: Rob Wultsch
Date:
Subject: Re: Issues with Quorum Commit