Re: Event Triggers reduced, v1 - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: Event Triggers reduced, v1
Date
Msg-id 1340064559-sup-8577@alvh.no-ip.org
Whole thread Raw
In response to Event Triggers reduced, v1  (Dimitri Fontaine <dfontaine@hi-media.com>)
Responses Re: Event Triggers reduced, v1
List pgsql-hackers
Excerpts from Dimitri Fontaine's message of vie jun 15 16:27:50 -0400 2012:

> The attached patch contains all the infrastructure for event triggers
> and also a first implementation of them for the event "command_start",
> implemented in a single place in utility.c.
>
> The infrastructure is about:
>
>  - new catalog
>  - grammar for new commands
>  - documentation skeleton
>  - pg_dump support
>  - psql support
>  - ability to actually run user triggers
>  - written in "core languages"
>     (pl/c, pl/pgsql, pl/python, pl/perl, pl/tcl)
>  - limited subcommand handling

Did you try REASSIGN OWNED and DROP OWNED with a role that has defined
some event triggers?

> Look, it's an easy little skinny patch to review, right:
>
> git --no-pager diff --shortstat master
>  62 files changed, 4546 insertions(+), 108 deletions(-)

Skinny ... right.  I started to give it a look -- I may have something
useful to comment later.

> This patch includes regression tests that we worked on with Thom last
> rounds, remember that they only run in the serial schedule, that means
> with `make installcheck` only. Adding noisy output at random while the
> parallel schedule run is a good way to break all the regression testing,
> so I've been avoiding that.

Hmm, I don't like the idea of a test that only runs in serial mode.
Maybe we can find some way to make it work in parallel mode as well.
I don't have anything useful to comment right now.

--
Álvaro Herrera <alvherre@commandprompt.com>
The PostgreSQL Company - Command Prompt, Inc.
PostgreSQL Replication, Consulting, Custom Development, 24x7 support


pgsql-hackers by date:

Previous
From: Christopher Browne
Date:
Subject: Re: [PATCH 10/16] Introduce the concept that wal has a 'origin' node
Next
From: Misa Simic
Date:
Subject: Re: [PATCH] Support for foreign keys with arrays