Re: Patch to handle connection limit for roles

Поиск
Список
Период
Сортировка
От Guillaume Lelarge
Тема Re: Patch to handle connection limit for roles
Дата
Msg-id 495B6D27.3050208@lelarge.info
обсуждение исходный текст
Ответ на Re: Patch to handle connection limit for roles  ("Dave Page" <dpage@pgadmin.org>)
Список pgadmin-hackers
Dave Page a écrit :
> On Wed, Dec 31, 2008 at 11:00 AM, Guillaume Lelarge
> <guillaume@lelarge.info> wrote:
>> Dave Page a écrit :
>>> On Tue, Dec 30, 2008 at 6:36 PM, Guillaume Lelarge
>>> <guillaume@lelarge.info> wrote:
>>>> Hi,
>>>>
>>>> This patch handles the connection limit parameter for roles. This
>>>> parameter is available since 8.1 but apparently, pgAdmin doesn't handle
>>>> this yet.
>>> HI,
>>>
>>> Looks good to the eyeballs, except that as far as I can see, the patch
>>> assumes the user is always using 8.1 or above and will not play nicely
>>> with 8.0. I think you need to add a liberal smattering of 'if
>>> (conn->BackendMinimumVersion(8, 1))'.
>>>
>> I assumed that pgRole and dlgRole are only used with roles (so
>> PostgreSQL 8.1 at least), and that pgUser and dlgUser are used with
>> older PostgreSQL releases. I just check this and my patch works for me
>> on 8.0.
>>
>> Is there something I misunderstood ?
>
> Nope - pre-coffee brain failure on my part. Please go ahead and commit
> - sorry for the noise!
>

No problem :)

Thanks, I commited it.


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

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

Предыдущее
От: svn@pgadmin.org
Дата:
Сообщение: SVN Commit by guillaume: r7530 - in trunk/pgadmin3: . pgadmin/dlg pgadmin/include/schema pgadmin/schema pgadmin/ui
Следующее
От: Ashesh Vashi
Дата:
Сообщение: Patch: New Plan Node Type support in pg 8.4