Re: Memory contexts reset for trigger invocations

Hi,

trigger.c goes through some trouble to free the tuples returned by
trigger functions. There's plenty codepaths that look roughly like:
                if (oldtuple != newtuple && oldtuple != slottuple)
                        heap_freetuple(oldtuple);
                if (newtuple == NULL)
                {
                        if (trigtuple != fdw_trigtuple)
                                heap_freetuple(trigtuple);
                        return NULL;            /* "do nothing" */
                }

but we, as far as I can tell, do not reset the memory context in which
the trigger functions have been called.

Wouldn't it be better to reset an appropriate context after each
invocation? Yes, that'd require some care to manage the lifetime of
tuples returned by triggers, but that seems OK?