Skip site navigation (1) Skip section navigation (2)

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 (view raw or flat)
Thread:
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
wise. 

* 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           www.2ndQuadrant.com


In response to

Responses

pgsql-hackers by date

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

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group