| From: | Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr> |
|---|---|
| To: | Vik Fearing <vik(dot)fearing(at)dalibo(dot)com> |
| Cc: | PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: [PATCH] pg_sleep(interval) |
| Date: | 2013-09-22 12:17:37 |
| Message-ID: | alpine.DEB.2.02.1309221411380.18614@sto |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hello,
> There is no pg_sleep(text) function and the cast is unknown->double
> precision.
My mistake.
As I understand it, pg_sleep('12') currently works and would not anymore
once your patch is applied. That is the concern raised by Robert Haas.
>> ISTM that providing "pg_sleep(TEXT)" cleanly resolves the
>> upward-compatibility issue raised.
>
> I don't like this idea at all. If we don't want to break compatibility
> for callers that quote the value, I would rather the patch be rejected
> outright.
That was just a suggestion, and I was trying to help. ISTM that if
Robert's concern is not addressed one way or another, you will just get
"rejected" on this basis.
--
Fabien.
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Sawada Masahiko | 2013-09-22 14:56:29 | Re: Patch for fail-back without fresh backup |
| Previous Message | Vik Fearing | 2013-09-22 11:13:35 | Re: [PATCH] pg_sleep(interval) |