| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
| Cc: | Thom Brown <thom(at)linux(dot)com>, pgsql-bugs <pgsql-bugs(at)postgresql(dot)org> |
| Subject: | Re: string_agg delimiter having no effect with order by |
| Date: | 2010-08-04 17:04:46 |
| Message-ID: | 21728.1280941486@sss.pgh.pa.us |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-bugs pgsql-hackers |
Robert Haas <robertmhaas(at)gmail(dot)com> writes:
> On Wed, Aug 4, 2010 at 12:44 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> Robert Haas <robertmhaas(at)gmail(dot)com> writes:
>>> I suppose this confusion is only possible because string_agg has both
>>> a one-argument and a two-argument form.
>>
>> Right, or at least that's what allows the mistake to go through without
>> reporting any error.
> No, that's what lets the correct form go through without reporting any error.
Really? IMO the reason Thom had a problem was he thought he was
invoking the two-argument form of string_agg, but he was really
invoking the one-argument form.
If we were a bit earlier in the 9.0 cycle I would suggest that this
confusion is a sufficient reason to drop the one-argument form of
string_agg. It's too late now though.
regards, tom lane
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2010-08-04 17:06:27 | Re: BUG #5598: Compatibility modes |
| Previous Message | Alex Hunsaker | 2010-08-04 16:51:44 | Re: BUG #5598: Compatibility modes |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Jan Otto | 2010-08-04 17:32:58 | patch for contrib/isn |
| Previous Message | Tom Lane | 2010-08-04 16:55:22 | Re: reducing NUMERIC size for 9.1 |