Fix AfterTriggerSaveEvent to use a test and elog, not just Assert, to check

Tom Lane <tgl@sss.pgh.pa.us>

Commit: 08dd656b71202911be7bcf4f554558a18138c867
Author: Tom Lane <tgl@sss.pgh.pa.us>
Date: 2009-10-27T20:14:56Z
Releases: 8.1.19
Fix AfterTriggerSaveEvent to use a test and elog, not just Assert, to check
that it's called within an AfterTriggerBeginQuery/AfterTriggerEndQuery pair.
The RI cascade triggers suppress that overhead on the assumption that they
are always run non-deferred, so it's possible to violate the condition if
someone mistakenly changes pg_trigger to mark such a trigger deferred.
We don't really care about supporting that, but throwing an error instead
of crashing seems desirable.  Per report from Marcelo Costa.

Files

PathChange+/−
src/backend/commands/trigger.c modified +8 −1