From: | "Robert Haas" <robertmhaas(at)gmail(dot)com> |
---|---|
To: | "Peter Eisentraut" <peter_e(at)gmx(dot)net> |
Cc: | "Jeff Davis" <pgsql(at)j-davis(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: array_agg and array_accum (patch) |
Date: | 2008-11-14 03:40:26 |
Message-ID: | 603c8f070811131940n4e71cf4oabdee2154da6b95f@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
It looks to me like section 34.10 of the docs might benefit from some
sort of update in light of this patch, since the builtin array_agg now
does the same thing as the proposed user-defined array_accum, only
better. Presumably we should either pick a different example, or add
a note that a builtin is available that does the same thing more
efficiently.
...Robert
On Thu, Nov 13, 2008 at 11:07 AM, Peter Eisentraut <peter_e(at)gmx(dot)net> wrote:
> Jeff Davis wrote:
>>
>> Here's an updated patch for just array_accum() with some simple docs.
>
> I have committed a "best of Robert Haas and Jeff Davis" array_agg() function
> with standard SQL semantics. I believe this gives the best consistency with
> other aggregate functions for the no-input-rows case. If some other behavior
> is wanted, it is a coalesce() away, as the documentation states.
>
> --
> Sent via pgsql-hackers mailing list (pgsql-hackers(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-hackers
>
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2008-11-14 03:46:11 | Re: Simple postgresql.conf wizard |
Previous Message | Robert Haas | 2008-11-14 03:34:50 | Re: Updated posix fadvise patch v19 |