>>>>> "Tom" == Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)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)