Re: Review: listagg aggregate - Mailing list pgsql-hackers

From Tom Lane
Subject Re: Review: listagg aggregate
Date
Msg-id 15536.1264694205@sss.pgh.pa.us
Whole thread Raw
In response to Re: Review: listagg aggregate  (Robert Haas <robertmhaas@gmail.com>)
Responses Re: Review: listagg aggregate  (Robert Haas <robertmhaas@gmail.com>)
Re: Review: listagg aggregate  (Greg Stark <stark@mit.edu>)
List pgsql-hackers
Robert Haas <robertmhaas@gmail.com> writes:
> On Thu, Jan 28, 2010 at 9:01 AM, Pavel Stehule <pavel.stehule@gmail.com> wrote:
>> simplest could not be a best. There have to be only a const
>> expression. But we have not possibility to check it in pg.

> Well... that's an entirely arbitrary limitation.  I admit that it
> doesn't seem likely that someone would want to have a variable
> delimiter, but putting extra effort and code complexity into
> preventing it seems pointless.

Yeah.  The real issue here is that in some cases you'd like to have
non-aggregated parameters to an aggregate, but SQL has no notation
to express that.

I think Pavel's underlying complaint is that if the delimiter
argument isn't constant, then we're exposing an implementation
dependency in terms of just which values get separated by which
delimiters.  The most practical implementation seems to be that
the first-call delimiter isn't actually used at all, and on
subsequent calls the delimiter *precedes* the associated value,
which is a bit surprising given the order in which one writes
them.  Not sure if this is worth documenting though.  Those two
or three people who actually try it will figure it out soon enough.
        regards, tom lane


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Streaming replication, and walsender during recovery
Next
From: Robert Haas
Date:
Subject: Re: Review: listagg aggregate