From: | Stephan Szabo <sszabo(at)megazone(dot)bigpanda(dot)com> |
---|---|
To: | Achilleus Mantzios <achill(at)matrix(dot)gatewaynet(dot)com> |
Cc: | Owen Jacobson <ojacobson(at)osl(dot)com>, pgsql-sql(at)postgresql(dot)org |
Subject: | Re: After Trigger assignment to NEW |
Date: | 2006-02-25 14:40:44 |
Message-ID: | 20060225062938.W75421@megazone.bigpanda.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-sql |
On Sat, 25 Feb 2006, Achilleus Mantzios wrote:
> O Owen Jacobson Feb 24, 2006 :
>
> > Achilleus Mantzios wrote:
> >
> > > O Tom Lane Feb 24, 2006 :
> > >
> > > > By definition, an AFTER trigger is too late to change what was
> > > > stored. Use a BEFORE trigger.
> > >
> > > Too late if someone wants to store it.
> > > I wanna store the intented original values, thats why i use
> > > AFTER trigger.
> > > But i would like to alter what a final AFTER trigger would see.
> > >
> > > I'll elabarote a little.
> > >
> > > An update happens.
> > > The row is stored.
> > > An after trigger is fired that alters some NEW columns
> > > (nullifies them), aiming for a subsequent trigger
> > > to see the altered results .
> > >
> > > It should be something like a pointer to a HeapTuple, (right?),
> > > so that would be feasible i suppose.
> > >
> > > I would not even make a post if it was something that trivial.
> > >
> > > I hope you get my point.
> >
> > Your real problem is that the "subsequent" trigger has behaviour you
> > don't like. That's what you should be fixing. If dbmirror has no way
> > to exclude specific tables from mirroring, take it up with them as a
> > feature request, or patch dbmirror to work how you want it to.
> >
> > AFTER triggers *must* receive the row that was actually
> > inserted/updated/deleted. If they could receive a "modified" row that
> > didn't reflect what was actually in the database, all sorts of useful
> > trigger-based logging and replication patterns wouldn't work, and
> > there's really no other way to implement them. See also Tom Lane's
> > other message for further implications of being able to modify the
> > rows seen by AFTER triggers.
> >
>
> As i have explained my dbmirror is FK null values gnostic(=aware) already
> as we speak.
[...]
> So nullifying a value just before the dbmirror trigger would do exactly
> the right thing (for me)
Yes it does what you want for this very specific case. But, would it do
what you want if someone put a trigger before it that changed the values
to some non-NULL thing? That seems likely to break your mirroring.
> > I'd also be hesitant to write triggers that have to execute in a specific order.
>
> Meaning that would hurt portability?
> Most people need features rathen than the relief to know they can migrate
> to another database (which they probably never will)
In this case, you're giving up the "feature" that users can write
constraints, logging or mirroring after triggers that are guaranteed to
get the data that was actually inserted in order to get the feature that a
trigger can affect the data to the next trigger. This seems like a general
loss in functionality for a larger fraction of users than those who gain.
> Back to AFTER trigger changing values issue,
> i think things are not so dramatic if
> FK triggers could just be fired first.
Actually, I think we technically fire the checks too early as it is, so I
don't see enshrining that or making it earlier is a good idea.
From | Date | Subject | |
---|---|---|---|
Next Message | Alvaro Herrera | 2006-02-25 15:24:16 | Re: Relation 0 does not exist |
Previous Message | Florian Weimer | 2006-02-25 14:17:41 | Index to support LIKE '%suffix' queries |