From e649796f128bd8702ba5744d36f4e8cb81f0b754 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Fri, 6 Jun 2003 15:04:03 +0000 Subject: Implement outer-level aggregates to conform to the SQL spec, with extensions to support our historical behavior. An aggregate belongs to the closest query level of any of the variables in its argument, or the current query level if there are no variables (e.g., COUNT(*)). The implementation involves adding an agglevelsup field to Aggref, and treating outer aggregates like outer variables at planning time. --- src/backend/executor/nodeAgg.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) (limited to 'src/backend/executor/nodeAgg.c') diff --git a/src/backend/executor/nodeAgg.c b/src/backend/executor/nodeAgg.c index 603df5ed1c..f2499cb4e5 100644 --- a/src/backend/executor/nodeAgg.c +++ b/src/backend/executor/nodeAgg.c @@ -45,7 +45,7 @@ * Portions Copyright (c) 1994, Regents of the University of California * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/executor/nodeAgg.c,v 1.105 2003/05/30 20:23:10 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/executor/nodeAgg.c,v 1.106 2003/06/06 15:04:01 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -1179,6 +1179,9 @@ ExecInitAgg(Agg *node, EState *estate) Datum textInitVal; int i; + /* Planner should have assigned aggregate to correct level */ + Assert(aggref->agglevelsup == 0); + /* Look for a previous duplicate aggregate */ for (i = 0; i <= aggno; i++) { -- cgit v1.2.1