| From: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
|---|---|
| To: | Itagaki Takahiro <itagaki(dot)takahiro(at)gmail(dot)com> |
| Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Takahiro Itagaki <itagaki(dot)takahiro(at)oss(dot)ntt(dot)co(dot)jp>, 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-07-12 20:40:27 |
| Message-ID: | AANLkTin1R0jYklt2XbiODiko-XDOTFmY_iOfAtYWsszJ@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hello
so this is actualised patch:
* concat_sql removed
* left, right, reverse and concat are in core
* printf and concat_ws are in contrib
* format show "<NULL>" as NULL string
* removed an using of wide chars
todo:
NULL handling for printf function
Query:
what is corect result for
* printf(">>%3.2d<<", NULL) ??
* printf(">>%3.2s", NULL) ??
Regards
Pavel Stehule
2010/7/12 Itagaki Takahiro <itagaki(dot)takahiro(at)gmail(dot)com>:
> 2010/7/12 Robert Haas <robertmhaas(at)gmail(dot)com>:
>> I'm all in favor of putting such things in core as are supported by
>> multiple competing products, but is that really true for all of these?
>
> - concat() : MySQL, Oracle, DB2
> - concat_ws() : MySQL,
> - left(), right() : MySQL, SQL Server, DB2
> - reverse() : MySQL, SQL Server, Oracle (as utl_raw.reverse)
>
> concat_sql(), format(), and sprintf() will be our unique features.
>
> --
> Itagaki Takahiro
>
| Attachment | Content-Type | Size |
|---|---|---|
| stringfunc.diff | application/octet-stream | 39.6 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Greg Smith | 2010-07-12 21:16:26 | Re: gSoC - ADD MERGE COMMAND - code patch submission |
| Previous Message | Joshua D. Drake | 2010-07-12 20:32:14 | Re: gSoC - ADD MERGE COMMAND - code patch submission |