>>>>> "Tom" == Tom Lane <t...@sss.pgh.pa.us> writes:

 Tom> Another approach would be to remove AggGetPerAggEContext as such
 Tom> from the API altogether, and instead offer an interface that
 Tom> says "register an aggregate cleanup callback", leaving it to the
 Tom> agg/window core code to figure out which context to hang that
 Tom> on.  I had thought that exposing this econtext and the
 Tom> per-input-tuple one would provide useful generality, but maybe
 Tom> we should rethink that.

 >> Works for me.

 Tom> If we're going to do that, I think it needs to be in 9.4.  Are
 Tom> you going to work up a patch?

Do we want a decision on the fn_extra matter first, or shall I do one
patch for the econtext, and a following one for fn_extra?

-- 
Andrew (irc:RhodiumToad)


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to