Re: Event Triggers: adding information

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Bruce Momjian <bruce(at)momjian(dot)us>
Cc: Dimitri Fontaine <dimitri(at)2ndquadrant(dot)fr>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Event Triggers: adding information
Date: 2013-01-23 20:02:24
Message-ID: CA+Tgmobzi9pDqWAtx_QfVq=JnP28kxW9camcuv3SGY=79D4nSQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Jan 23, 2013 at 2:36 PM, Bruce Momjian <bruce(at)momjian(dot)us> wrote:
> On Wed, Jan 23, 2013 at 09:33:58AM -0500, Robert Haas wrote:
>> On Wed, Jan 23, 2013 at 4:57 AM, Dimitri Fontaine
>> <dimitri(at)2ndquadrant(dot)fr> wrote:
>> > Thanks for commiting the fixes. About the regression tests, I think
>> > you're right, but then I can't see how to include such a test. Maybe you
>> > could add the other one, though?
>>
>> Can you point me specifically at what you have in mind so I can make
>> sure I'm considering the right thing?
>>
>> > +1 for this version, thanks.
>>
>> OK, committed that also.
>
> Also, I assume we no longer want after triggers on system tables, so I
> removed that from the TODO list and added event triggers as a completed
> item.

Seems reasonable. Event triggers are not completed in the sense that
there is a lot more stuff we can do with this architecture, but we've
got a basic implementation now and that's progress. And they do
address the use case that triggers on system tables would have
targeted, I think, but better.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2013-01-23 20:04:40 Re: COPY FREEZE has no warning
Previous Message Bruce Momjian 2013-01-23 19:51:01 Re: [PATCH] pg_isready (was: [WIP] pg_ping utility)