A subplan invoked within an aggregate function's argument should

be allowed to receive ungrouped variables of the current query level.
Curious that no one reported this bug before...

diff --git a/src/backend/optimizer/util/clauses.c b/src/backend/optimizer/util/clauses.c
index e6d51b047ccfa7009e640cbe5f36e4bc2ddcfdf0..bc05908148b372b205f17a561725eb60d630218f 100644 (file)
--- a/src/backend/optimizer/util/clauses.c
+++ b/src/backend/optimizer/util/clauses.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/optimizer/util/clauses.c,v 1.81 2001/02/12 18:46:40 momjian Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/optimizer/util/clauses.c,v 1.82 2001/03/08 01:49:01 tgl Exp $
  *
  * HISTORY
  *       AUTHOR                        DATE                    MAJOR EVENT
@@ -542,6 +542,14 @@ check_subplans_for_ungrouped_vars_walker(Node *node,
        if (node == NULL)
                return false;
 
+       /*
+        * If we find an aggregate function, do not recurse into its
+        * arguments.  Subplans invoked within aggregate calls are allowed
+        * to receive ungrouped variables.
+        */
+       if (IsA(node, Aggref))
+               return false;
+
        /*
         * We can ignore Vars other than in subplan args lists, since the
         * parser already checked 'em.