From: | Kevin Grittner <kgrittn(at)ymail(dot)com> |
---|---|
To: | Simon Riggs <simon(at)2ndQuadrant(dot)com>, Craig Ringer <craig(at)2ndquadrant(dot)com> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Batch API for After Triggers |
Date: | 2013-06-17 19:53:45 |
Message-ID: | 1371498825.21517.YahooMailNeo@web162905.mail.bf1.yahoo.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Simon Riggs <simon(at)2ndQuadrant(dot)com> wrote:
> On 9 June 2013 12:58, Craig Ringer <craig(at)2ndquadrant(dot)com> wrote:
>> We don't currently have OLD and NEW relations so we're free to
>> define how this works pretty freely.
> I think the best way, if we did do this, would be to have a
> number of different relations defined:
>
> OLD
> NEW
> INSERTED
> DELETED
> all of which would be defined same as main table
>
> and also one called
> UPDATED
> which would have two row vars called OLD and NEW
> so you would access it like e.g. IF UPDATED.OLD.id = 7
Well, there is the SQL standard, which has a couple paragraphs on
the topic which we might want to heed. For a delete there is just
an old table; for an insert just a new one. For an update you have
both, with the same cardinality. The rows in the old and new
tables have a correspondence, but that is only visible to FOR EACH
ROW triggers. For something like RI, why would you need to
establish correspondence? A row with the referenced key either
exists after the statement completes, or it doesn't -- why would we
care whether it is an updated version of the same row?
Syntax for how to refer to the these is defined by the standard.
As usual, I don't object to adding capabilities as long as the
standard syntax is also supported with standard semantics.
--
Kevin Grittner
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2013-06-17 20:00:05 | Re: Support for REINDEX CONCURRENTLY |
Previous Message | Josh Berkus | 2013-06-17 19:52:36 | Re: Support for REINDEX CONCURRENTLY |