Re: Review: listagg aggregate

From: "David E(dot) Wheeler" <david(at)kineticode(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Peter Eisentraut <peter_e(at)gmx(dot)net>, Scott Bailey <artacus(at)comcast(dot)net>, "pgsql-hackers(at)postgresql(dot)org Hackers" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Review: listagg aggregate
Date: 2010-01-26 18:04:23
Message-ID: B0A1647F-AF31-4D0C-BC29-26AA1EE9F2B1@kineticode.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Jan 26, 2010, at 9:36 AM, Tom Lane wrote:

> But what it *produces* is a string. For comparison, the
> SQL-standard-specified array_agg produces arrays, but what it
> acts on isn't an array.

Meh. This is all just bike-shedding. I'm fine with string_agg(), though in truth none of the names has really been great. The inclusion of "agg" in the name is unfortunate.

I'll have a look at Pavel's new patch now.

David

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2010-01-26 18:09:05 Re: Review: listagg aggregate
Previous Message Kevin Grittner 2010-01-26 17:39:19 Re: testing cvs HEAD - HS/SR - missing file