On Tue, Nov 10, 2009 at 2:56 PM, Marko Tiikkaja <marko.tiikk...@cs.helsinki.fi> wrote: > (Sorry, forgot to CC the list) > > Tom Lane wrote: >> >> This doesn't really seem like a good idea from here. You're changing >> a decision that has something like twenty years' standing in the code, >> for no real gain. AFAICS this is just going to move the special cases >> from point A to point B. > > Right, but this way you only have to special-case in grouping_planner(), > and targetList always means the same thing.
I haven't read through the patch but I can say the existing arrangement certainly seemed strange to me when I first saw it. -- greg -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
