OSDN Git Service

My first chosen victim for expression_tree_walker conversion
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 19 Jun 1999 03:48:31 +0000 (03:48 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 19 Jun 1999 03:48:31 +0000 (03:48 +0000)
commite786508600e15c33d0727374b466fa7854c7c77e
treee3802b59d236d42767b4145757f13da59ef6c392
parent86f36719dba6641b4437536ee54e647c12282332
My first chosen victim for expression_tree_walker conversion
is parse_aggs.c.  This fixes its failure to cope with (at least) CaseExpr
and ArrayRef nodes, which is the reason why both of these fail in 6.5:
select coalesce(f1,0) from int4_tbl group by f1;
ERROR:  Illegal use of aggregates or non-group column in target list
select sentence.words[0] from sentence group by sentence.words[0];
ERROR:  Illegal use of aggregates or non-group column in target list
The array case still fails, but at least it's not parse_agg's fault
anymore ... considering that we now support CASE officially, I think
it's important to fix the first example ...
src/backend/parser/parse_agg.c