diff --git a/src/backend/optimizer/util/clauses.c b/src/backend/optimizer/util/clauses.c index f6a943fcab4ac8671ba0871524eb1fc8d771ebb8..dec753b3a3dcff22868005405c4b501675a8ae5d 100644 --- a/src/backend/optimizer/util/clauses.c +++ b/src/backend/optimizer/util/clauses.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/optimizer/util/clauses.c,v 1.287 2010/03/19 22:54:41 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/optimizer/util/clauses.c,v 1.288 2010/08/14 15:47:13 tgl Exp $ * * HISTORY * AUTHOR DATE MAJOR EVENT @@ -549,6 +549,18 @@ count_agg_clauses_walker(Node *node, AggClauseCounts *counts) counts->transitionSpace += avgwidth + 2 * sizeof(void *); } + else if (aggtranstype == INTERNALOID) + { + /* + * INTERNAL transition type is a special case: although INTERNAL + * is pass-by-value, it's almost certainly being used as a pointer + * to some large data structure. We assume usage of + * ALLOCSET_DEFAULT_INITSIZE, which is a good guess if the data is + * being kept in a private memory context, as is done by + * array_agg() for instance. + */ + counts->transitionSpace += ALLOCSET_DEFAULT_INITSIZE; + } /* * Complain if the aggregate's arguments contain any aggregates;