Re: Set statement timeout in the query tool

Поиск
Список
Период
Сортировка
От Guillaume Lelarge
Тема Re: Set statement timeout in the query tool
Дата
Msg-id 200912041855.02863.guillaume@lelarge.info
обсуждение исходный текст
Ответ на Re: Set statement timeout in the query tool  (Dave Page <dpage@pgadmin.org>)
Список pgadmin-hackers
Le vendredi 4 décembre 2009 à 18:43:47, Dave Page a écrit :
> On Fri, Dec 4, 2009 at 5:33 PM, Guillaume Lelarge
>
> <guillaume@lelarge.info> wrote:
> > I see a use case for having both. If someone uses a lot the
> > statement_timeout option, he'll be annoyed to put it in the text box
> > every time he launches the query tool.
>
> Just save the value in the toolbar using sysSettings.
>

Oh yes, I didn't think about that. Always thought that settings should all be
in the Options window, even if it's possible to set them also elsewhere
(examples are settings for the query tool).

> >> Why save/reset it? The next query will just change it again anyway,
> >> and there's no way round that.
> >
> > Because every other query launch by the query tool will use it. For
> > example the one of the GQB (which, AFAIK, uses the same connection).
>
> Right - but you'd still only need to set it once, when the window is
> opened, or if the value is changed. There's no need to keep resetting
> it.
>

Right too.

> > I don't have any issues with adding a UI for other parameters (work_mem
> > comes to mind immediately). We can put them all in their own toolbar, so
> > that users can hide it, so that it doesn't cluter the window UI.
>
> I do. Extra code maintenance and complexity for little gain.
>

Yes, sure. It's not really a complex functionality, but many minors make a
complex one.

I've asked Fabrice to give me more details. If it's more interesting this
time, I'll probably come back with another (better?) patch.


--
Guillaume.
 http://www.postgresqlfr.org
 http://dalibo.com

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

Предыдущее
От: Dave Page
Дата:
Сообщение: Re: Set statement timeout in the query tool
Следующее
От: svn@pgadmin.org
Дата:
Сообщение: SVN Commit by guillaume: r8112 - in trunk/www/locale: cs_CZ/LC_MESSAGES fr_FR/LC_MESSAGES zh_CN/LC_MESSAGES