Re: WITHIN GROUP patch

>>>>> "Tom" == Tom Lane <tgl@sss.pgh.pa.us> writes:
Tom> After examining this more closely, ISTM that the directTom> arguments are supposed to be processed as if they
weren'tinsideTom> an aggregate call at all.  That being the case, isn't it flatTom> out wrong for check_agg_arguments()
tobe examining theTom> agg_ordset list?  It should ignore those expressions whilstTom> determining the aggregate's
semanticlevel.  As an example, anTom> upper-level Var in those expressions isn't grounds for decidingTom> that the
aggregateisn't of the current query level.
 

Hmm... yes, you're probably right; but we'd still have to check somewhere
for improper nesting, no? since not even the direct args are allowed to
contain nested aggregate calls.

-- 
Andrew (irc:RhodiumToad)