Re: [PATCH] pg_sleep(interval) - Mailing list pgsql-hackers

From Fabien COELHO
Subject Re: [PATCH] pg_sleep(interval)
Date
Msg-id alpine.DEB.2.02.1309221411380.18614@sto
Whole thread Raw
In response to Re: [PATCH] pg_sleep(interval)  (Vik Fearing <vik.fearing@dalibo.com>)
Responses Re: [PATCH] pg_sleep(interval)
List 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.



pgsql-hackers by date:

Previous
From: Vik Fearing
Date:
Subject: Re: [PATCH] pg_sleep(interval)
Next
From: Sawada Masahiko
Date:
Subject: Re: Patch for fail-back without fresh backup