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

From Pavel Stehule
Subject Re: Review: listagg aggregate
Date
Msg-id 162867791001312107le140d91wbe1cf29669ef76da@mail.gmail.com
Whole thread Raw
In response to Re: Review: listagg aggregate  (Takahiro Itagaki <itagaki.takahiro@oss.ntt.co.jp>)
Responses Re: Review: listagg aggregate
List pgsql-hackers
2010/2/1 Takahiro Itagaki <itagaki.takahiro@oss.ntt.co.jp>:
>
> Robert Haas <robertmhaas@gmail.com> wrote:
>
>> > ok - I am only one who like original behave - so I ?am withdrawing.
>> > Robert, If you like, please commit Itagaki's patch. + add note about
>> > behave when second parameter isn't stable.
>>
>> I haven't even looked at this code - I sort of assumed Itagaki was
>> handling this one.  But it might be good to make sure that the docs
>> have been read through by a native English speaker prior to commit...
>
> Applied with some editorialization. Please check the docs are good enough.
>
> I removed a test pattern for variable delimiters from regression test
> because it might be an undefined behavior. We might change the behavior
> in the future, so we guarantee only constant delimiters.
>
> The transition functions are renamed to "string_agg_transfn" and
> "string_agg_delim_transfn". We cannot use "string_agg_transfn" for
> both names because we cast the function name as regproc in bootstrap;
> it complains about duplicated function names.

thank you very much

Pavel Stehule

>
> Regards,
> ---
> Takahiro Itagaki
> NTT Open Source Software Center
>
>
>


pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: Hot Standby and VACUUM FULL
Next
From: Takahiro Itagaki
Date:
Subject: Re: Largeobject Access Controls (r2460)