From: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
---|---|
To: | Itagaki Takahiro <itagaki(dot)takahiro(at)gmail(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Robert Haas <robertmhaas(at)gmail(dot)com>, Merlin Moncure <mmoncure(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Erik Rijkers <er(at)xs4all(dot)nl> |
Subject: | Re: patch (for 9.1) string functions |
Date: | 2010-08-24 06:38:24 |
Message-ID: | AANLkTi=a12zMEML2eXh7ADA9caDteaYYbhAD91ieYY7D@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
2010/8/24 Itagaki Takahiro <itagaki(dot)takahiro(at)gmail(dot)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(dot)stehule(at)gmail(dot)com> wrote:
>> 2010/8/23 Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)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
>
From | Date | Subject | |
---|---|---|---|
Next Message | Kevin Grittner | 2010-08-24 07:03:56 | Re: [Glue] Deadlock bug |
Previous Message | Itagaki Takahiro | 2010-08-24 06:32:47 | Re: patch (for 9.1) string functions |