>>>>> "Tom" == Tom Lane <tgl@sss.pgh.pa.us> writes:
Tom> Hm, I was just working on inserting something of the sort intoTom> ExecInitAgg. But I guess we could do it in the
plannertoo. WillTom> run with your approach.
Tom> I think it's a bit too stupid as-is, though. We don't need toTom> 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)