From 763ff8aef848d71da079049890786edffc3302d6 Mon Sep 17 00:00:00 2001 From: Bruce Momjian Date: Thu, 15 Jan 1998 19:00:16 +0000 Subject: Remove Query->qry_aggs and qry_numaggs and replace with Query->hasAggs. Pass List* of Aggregs into executor, and create needed array there. No longer need to double-processs Aggregs with second copy in Query. Fix crash when doing: select sum(x+1) from test where 1 > 0; --- src/backend/parser/analyze.c | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) (limited to 'src/backend/parser/analyze.c') diff --git a/src/backend/parser/analyze.c b/src/backend/parser/analyze.c index dcb9eecb5d..45c5d1406b 100644 --- a/src/backend/parser/analyze.c +++ b/src/backend/parser/analyze.c @@ -7,7 +7,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/parser/analyze.c,v 1.64 1998/01/11 03:41:35 momjian Exp $ + * $Header: /cvsroot/pgsql/src/backend/parser/analyze.c,v 1.65 1998/01/15 18:59:56 momjian Exp $ * *------------------------------------------------------------------------- */ @@ -219,7 +219,7 @@ transformDeleteStmt(ParseState *pstate, DeleteStmt *stmt) qry->resultRelation = refnameRangeTablePosn(pstate->p_rtable, stmt->relname); /* make sure we don't have aggregates in the where clause */ - if (pstate->p_numAgg > 0) + if (pstate->p_hasAggs) parseCheckAggregates(pstate, qry); return (Query *) qry; @@ -334,7 +334,7 @@ transformInsertStmt(ParseState *pstate, InsertStmt *stmt) qry->targetList, qry->uniqueFlag); - if (pstate->p_numAgg > 0) + if (pstate->p_hasAggs) finalizeAggregates(pstate, qry); qry->unionall = stmt->unionall; /* in child, so unionClause may be false */ @@ -796,8 +796,7 @@ transformRuleStmt(ParseState *pstate, RuleStmt *stmt) pstate->p_last_resno = 1; pstate->p_is_rule = true; /* for expand all */ - pstate->p_numAgg = 0; - pstate->p_aggs = NULL; + pstate->p_hasAggs = false; lfirst(actions) = transformStmt(pstate, lfirst(actions)); actions = lnext(actions); @@ -853,7 +852,7 @@ transformSelectStmt(ParseState *pstate, SelectStmt *stmt) qry->targetList); qry->rtable = pstate->p_rtable; - if (pstate->p_numAgg > 0) + if (pstate->p_hasAggs) finalizeAggregates(pstate, qry); qry->unionall = stmt->unionall; /* in child, so unionClause may be false */ @@ -890,11 +889,11 @@ transformUpdateStmt(ParseState *pstate, UpdateStmt *stmt) qry->rtable = pstate->p_rtable; qry->resultRelation = refnameRangeTablePosn(pstate->p_rtable, stmt->relname); - if (pstate->p_numAgg > 0) + if (pstate->p_hasAggs) finalizeAggregates(pstate, qry); /* make sure we don't have aggregates in the where clause */ - if (pstate->p_numAgg > 0) + if (pstate->p_hasAggs) parseCheckAggregates(pstate, qry); return (Query *) qry; -- cgit v1.2.1