From: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
---|---|
To: | Neil Conway <neilc(at)samurai(dot)com> |
Cc: | PostgreSQL Patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: per-statement triggers |
Date: | 2002-11-22 05:41:21 |
Message-ID: | 200211220541.gAM5fLX10550@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
Your patch has been added to the PostgreSQL unapplied patches list at:
http://momjian.postgresql.org/cgi-bin/pgpatches
I will try to apply it within the next 48 hours.
---------------------------------------------------------------------------
Neil Conway wrote:
> This patch implements FOR EACH STATEMENT triggers, per my email to
> -hackers a couple days ago.
>
> Notes/caveats:
>
> - added regression tests for the new functionality, all
> regression tests pass on my machine
>
> - added pg_dump support
>
> - updated PL/PgSQL to support per-statement triggers; didn't
> look at the other procedural languages.
>
> - there's (even) more code duplication in trigger.c than there
> was previously. Any suggestions on how to refactor the
> ExecXXXTriggers() functions to reuse more code would be
> welcome -- I took a brief look at it, but couldn't see an
> easy way to do it (there are several subtly-different
> versions of the code in question)
>
> - updated the documentation. I also took the liberty of
> removing a big chunk of duplicated syntax documentation in
> the Programmer's Guide on triggers, and moving that
> information to the CREATE TRIGGER reference page.
>
> - I also included some spelling fixes and similar small
> cleanups I noticed while making the changes. If you'd like
> me to split those into a separate patch, let me know.
>
> Cheers,
>
> Neil
>
> --
> Neil Conway <neilc(at)samurai(dot)com> || PGP Key ID: DB3C29FC
[ Attachment, skipping... ]
>
> ---------------------------(end of broadcast)---------------------------
> TIP 5: Have you checked our extensive FAQ?
>
> http://www.postgresql.org/users-lounge/docs/faq.html
--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2002-11-22 05:41:31 | Re: documentation fix |
Previous Message | Bruce Momjian | 2002-11-22 05:41:01 | Re: 7.4 cube patch to add create cast |