>>>>> "Tom" == Tom Lane <t...@sss.pgh.pa.us> writes:
Tom> Hm, I was just working on inserting something of the sort into Tom> ExecInitAgg. But I guess we could do it in the planner too. Will Tom> run with your approach. Tom> I think it's a bit too stupid as-is, though. We don't need to Tom> recalculate for Params in aggdirectargs, do we? In theory we would need to. But in practice we don't, because we don't allow ordered aggs in AGG_HASHED mode anyway. We could skip filling in aggParam at all if not in AGG_HASHED mode I guess. -- 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