Re: [HACKERS] [BUGS] BUG #5608: array_agg() consumes too much memory

2010-08-14 Thread Tom Lane
Hitoshi Harada writes: > 2010/8/14 Itagaki Takahiro : >> The attached patch is the near-term fix; it adds ALLOCSET_DEFAULT_INITSIZE >> bytes to memory assumption. >> >> We might need the same adjustment for string_agg(), that consumes >> 1024 bytes for the transit condition. array_agg() and strin

Re: [HACKERS] [BUGS] BUG #5608: array_agg() consumes too much memory

2010-08-14 Thread Tom Lane
Hitoshi Harada writes: > 2010/8/14 Itagaki Takahiro : >> 2010/8/10 Tom Lane : >>> Eventually it might be nice to have some sort of way to specify the >>> estimate to use for any aggregate function --- but for a near-term >>> fix maybe we should just hard-wire a special case for array_agg in >>> co

Re: [HACKERS] [BUGS] BUG #5608: array_agg() consumes too much memory

2010-08-13 Thread Hitoshi Harada
2010/8/14 Itagaki Takahiro : > 2010/8/10 Tom Lane : >> Eventually it might be nice to have some sort of way to specify the >> estimate to use for any aggregate function --- but for a near-term >> fix maybe we should just hard-wire a special case for array_agg in >> count_agg_clauses_walker(). > > T

Re: [HACKERS] [BUGS] BUG #5608: array_agg() consumes too much memory

2010-08-13 Thread Itagaki Takahiro
2010/8/10 Tom Lane : > Eventually it might be nice to have some sort of way to specify the > estimate to use for any aggregate function --- but for a near-term > fix maybe we should just hard-wire a special case for array_agg in > count_agg_clauses_walker(). The attached patch is the near-term fix