Re: [PATCH] pg_sleep(interval)

Поиск
Список
Период
Сортировка
От Fabien COELHO
Тема Re: [PATCH] pg_sleep(interval)
Дата
Msg-id alpine.DEB.2.02.1309221411380.18614@sto
обсуждение исходный текст
Ответ на Re: [PATCH] pg_sleep(interval)  (Vik Fearing <vik.fearing@dalibo.com>)
Ответы Re: [PATCH] pg_sleep(interval)  (Vik Fearing <vik.fearing@dalibo.com>)
Список 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 по дате отправления:

Предыдущее
От: Vik Fearing
Дата:
Сообщение: Re: [PATCH] pg_sleep(interval)
Следующее
От: Sawada Masahiko
Дата:
Сообщение: Re: Patch for fail-back without fresh backup