Re: patch to add \watch to psql

Поиск
Список
Период
Сортировка
От Will Leinweber
Тема Re: patch to add \watch to psql
Дата
Msg-id CAL8LqZS0HFx_vNp+_--xJL_1XLiYYKguhDdtKjt-+kJ5wyB38g@mail.gmail.com
обсуждение исходный текст
Ответ на Re: patch to add \watch to psql  (Peter Eisentraut <peter_e@gmx.net>)
Ответы Re: patch to add \watch to psql  ("Dickson S. Guedes" <listas@guedesoft.net>)
Re: patch to add \watch to psql  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
Here is an updated patch that addresses several of the points brought up so far, such as the sleep, internationalization banner, and zero wait check, and it removes the premature input check.

Unfortunately rl_clear_screen() is not included at all in libedit, causing compilation to fail, and I was completely unable to find a way to distinguish libedit from readline on OS X. It tries extraordinarily hard to pretend that it's readline. Instead falling back to simple control characters to clear the screen worked very well on both linux and OS X.


On Tue, Mar 26, 2013 at 2:14 PM, Peter Eisentraut <peter_e@gmx.net> wrote:
On 3/24/13 3:10 PM, Tom Lane wrote:
> I also concur with the complaint here
> http://www.postgresql.org/message-id/CAAZKuFZxyj-RT1aeC6s0g7zM68tDLfbBM1R6HGrbbxnz80KcoA@mail.gmail.com
> that allowing a minimum sleep of 0 is rather dangerous

The original "watch" command apparently silently corrects a delay of 0
to 0.1 seconds.

> Another minor question is whether we really need the time-of-day in the
> banner,

That's also part of the original "watch" and occasionally useful, I think.


Вложения

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

Предыдущее
От: Gavin Flower
Дата:
Сообщение: Re: [PATCH] Exorcise "zero-dimensional" arrays (Was: Re: Should array_length() Return NULL)
Следующее
От: Tom Lane
Дата:
Сообщение: Re: spoonbill vs. -HEAD