On Wed, Dec 31, 2008 at 11:00 AM, Guillaume Lelarge
> Dave Page a écrit :
>> On Tue, Dec 30, 2008 at 6:36 PM, Guillaume Lelarge
>> <guillaume(at)lelarge(dot)info> wrote:
>>> This patch handles the connection limit parameter for roles. This
>>> parameter is available since 8.1 but apparently, pgAdmin doesn't handle
>>> this yet.
>> 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!
EnterpriseDB UK: http://www.enterprisedb.com
In response to
pgadmin-hackers by date
|Next:||From: Quan Zongliang||Date: 2008-12-31 11:18:12|
|Subject: Re: import function's progress report|
|Previous:||From: Guillaume Lelarge||Date: 2008-12-31 11:00:51|
|Subject: Re: Patch to handle connection limit for roles|