pgsql: Fix planner to make a reasonable assumption about the amount of

Log Message:
-----------
Fix planner to make a reasonable assumption about the amount of memory space
used by array_agg(), string_agg(), and similar aggregate functions that use
"internal" as their transition datatype. The previous coding thought this
took *no* extra space, since "internal" is pass-by-value; but actually these
aggregates typically consume a great deal of space. Per bug #5608 from
Itagaki Takahiro, and fix suggestion from Hitoshi Harada.

Back-patch to 8.4, where array_agg was introduced.

Tags:
----
REL9_0_STABLE

Modified Files:
--------------
pgsql/src/backend/optimizer/util:
clauses.c (r1.287 -> r1.287.4.1)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/optimizer/util/clauses.c?r1=1.287&r2=1.287.4.1)