Re: proposal - psql - use pager for \watch command
От | Tom Lane |
---|---|
Тема | Re: proposal - psql - use pager for \watch command |
Дата | |
Msg-id | 996255.1654572180@sss.pgh.pa.us обсуждение исходный текст |
Ответ на | Re: proposal - psql - use pager for \watch command (Thomas Munro <thomas.munro@gmail.com>) |
Ответы |
Re: proposal - psql - use pager for \watch command
|
Список | pgsql-hackers |
Thomas Munro <thomas.munro@gmail.com> writes: > On Mon, May 9, 2022 at 7:07 PM Pavel Stehule <pavel.stehule@gmail.com> wrote: >> út 13. 7. 2021 v 19:50 odesílatel Tom Lane <tgl@sss.pgh.pa.us> napsal: >>> ^C\watch cancelled >>> regression=# > Do we really need the extra text? What about just \n, so you get: > postgres=# \watch 1 > ...blah blah... > ^C > postgres=# Fine by me. > This affects all release branches too. Should we bother to fix this > there? For them, I think the fix is just: If we're doing something as nonintrusive as just adding a newline, it'd probably be OK to backpatch. The code needs a comment about why it's emitting a newline, though. In particular, it had better explain why that should be conditional on !pagerpipe, because that makes no sense to me. regards, tom lane
В списке pgsql-hackers по дате отправления: