Re: Review: listagg aggregate

From: Simon Riggs <simon(at)2ndQuadrant(dot)com>
To: "David E(dot)Wheeler" <david(at)kineticode(dot)com>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: Review: listagg aggregate
Date: 2010-01-24 18:02:25
Message-ID: 1264356145.10311.205.camel@ebony
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

On Fri, 2010-01-22 at 11:14 -0800, David E.Wheeler wrote:
> No performance issues

ISTM that this class of function is inherently dangerous performance

* It looks incredibly easy to construct enormous lists. We should test
the explosion limit of this to see how it is handled. Perhaps we need
some parameter limits to control that, depending upon results.

* Optimizer doesn't consider whether the result type of an aggregate get
bigger as the aggregate processes more rows. If we're adding this
function we should give some thought in that area also, or at least a
comment to note that it can and will cause the optimizer problems in
complex queries.

Simon Riggs

In response to


Browse pgsql-hackers by date

  From Date Subject
Next Message Euler Taveira de Oliveira 2010-01-24 18:04:04 Re: Resetting a single statistics counter
Previous Message Kevin Grittner 2010-01-24 17:56:49 Re: Largeobject Access Controls (r2460)