Re: patch: bytea_agg

From: Merlin Moncure <mmoncure(at)gmail(dot)com>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: Peter Eisentraut <peter_e(at)gmx(dot)net>, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: patch: bytea_agg
Date: 2011-12-24 13:13:32
Message-ID: CAHyXU0zcqh3smyGASbDJb=SVng-=HrWh=Od1LzofavpOCZjaHw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Dec 23, 2011 at 12:30 PM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> Well, because it doesn't operate on strings.
>
> I argued when we added string_agg that it ought to be called
> concat_agg, or something like that, but I got shouted down.  So now
> here we are.

+1. Using the input type names to name the function is a mistake and
should be stopped...enough. It's verbose and unnecessary...everything
else in sql is heavily overloaded (we don't have int_max() and
float_max()).

merlin

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2011-12-24 14:46:16 Re: 16-bit page checksums for 9.2
Previous Message Simon Riggs 2011-12-24 12:20:25 Re: Page Checksums + Double Writes