Re: patch (for 9.1) string functions

2010/8/24 Itagaki Takahiro <itagaki.takahiro@gmail.com>:
> I applied the attached patch to HEAD. concat(), concat_ws(), left(),
> right(), and reverse() are in it, but format() and sprintf() are not.
> It's my understanding that we don't have consensus about the best syntax
> for the formatting function. We can forget about RAISE. C-like printf
> syntax is the next candidate, but we should consider about other ones
> restarting with a clean slate.
>

Thank you very much

C-like printf function is the most worse candidate - I don't like to
repeat discussion again - this function is designed for totally
different environment than SQL. I am sure, so we don't need a function
with complex formatting - maintaining "to_char" is good example.

Regards

Pavel Stehule



> Anyway, the newly added functions are useful for developers especially
> migrated from other database products. Thank you.
>
>
> On Mon, Aug 23, 2010 at 11:41 PM, Pavel Stehule <pavel.stehule@gmail.com> wrote:
>> 2010/8/23 Tom Lane <tgl@sss.pgh.pa.us>:
>>> You should leave RAISE alone and just think about printf.
>>
>> ok - then we don't need modify proposed patch. "Format" function is
>> enough for PL/pgSQL and other PL languages has own mutation of this
>> functions. There are not barrier for implementation as custom
>> function, so we can hold this function most simple.
>
> --
> Itagaki Takahiro
>