projects / users / c2main / postgres.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: dd56a9d) | patch
Fix planner to make a reasonable assumption about the amount of memory space
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 14 Aug 2010 15:47:30 +0000 (15:47 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 14 Aug 2010 15:47:30 +0000 (15:47 +0000)
commit286fa734712f3d034ce402d712ba7291c1f64bac
tree019d81d7a3b7ee4e5605b79d2fccc16f2ce0c683tree
parentdd56a9d6ededebc25e8f57f9d3a58e1b94f42a81commit | diff
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.
src/backend/optimizer/util/clauses.c diff | blob | blame | history
Cedric's Stuff