Re: [PATCH] pg_sleep(interval)

Поиск
Список
Период
Сортировка
От Fabien COELHO
Тема Re: [PATCH] pg_sleep(interval)
Дата
Msg-id alpine.DEB.2.02.1310161051420.11284@sto
обсуждение исходный текст
Ответ на Re: [PATCH] pg_sleep(interval)  (Vik Fearing <vik.fearing@dalibo.com>)
Ответы Re: [PATCH] pg_sleep(interval)
Список pgsql-hackers
Hello Vik,

>> Yes, I understand you are trying to help, and I appreciate it!  My
>> opinion, and that of others as well from the original thread, is that
>> this patch should either go in as is and break that one case, or not go
>> in at all.  I'm fine with either (although clearly I would prefer it
>> went in otherwise I wouldn't have written the patch).
>
> I see this is marked as rejected in the commitfest app, but I don't see
> any note about who did it or why.  I don't believe there is consensus
> for rejection on this list. In fact I think the opposite is true.
>
> May we have an explanation please from the person who rejected this
> without comment?

I did it, on the basis that you stated that you prefered not adding 
pg_sleep(TEXT) to answer Robert Haas concern about preserving 
pg_sleep('10') current functionality, and that no other solution was 
suggested to tackle this issue.

If I'm mistaken, feel free to change the state back to what is 
appropriate.

My actual opinion is that breaking pg_sleep('10') is no big deal, but I'm 
nobody here, and Robert is somebody, so I think that his concern must be 
addressed.

-- 
Fabien.



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Ronan Dunklau
Дата:
Сообщение: Re: Triggers on foreign tables
Следующее
От: Vik Fearing
Дата:
Сообщение: Re: [PATCH] pg_sleep(interval)