Thread: Re: Newer server version warning (was: SVN Commit by dpage: r7544...)

Re: Newer server version warning (was: SVN Commit by dpage: r7544...)

From
"Dave Page"
Date:
On Mon, Jan 12, 2009 at 1:18 PM, Guillaume Lelarge
<guillaume@lelarge.info> wrote:
> Quan Zongliang a écrit :
>>>> Now, const float SERVER_MIN_VERSION = 7.3f; is defined in pgAdmin3.h.
>>>> Add a new variable SERVER_MAX_VERSION here.
>>>> When end-users try to connect to db, check the version.
>>>> If it is newer, display a warning dialog.
>>>> Then the end-users click <OK> button to continue.
>>
>> I think again. It's really not a good idea.
>> Another option is needed. Otherwise, when end users connect to db
>> with an older version pgAdmin. They will get message box always,
>> but the pgAdmin maybe work fine. This must make them very angry.
>>
>> Add a new option like "Don't check database version."?
>>
>
> I don't think so. pgAdmin should display the warning box only if they
> try to backup/restore, and not at startup time.

This is to prevent the user seeing errors when they run an old pgAdmin
against a newer server - it's not really about backup/restore (the
thread kinda changed subject).

--
Dave Page
EnterpriseDB UK:   http://www.enterprisedb.com

Re: Newer server version warning (was: SVN Commit by dpage: r7544...)

From
Guillaume Lelarge
Date:
Dave Page a écrit :
> On Mon, Jan 12, 2009 at 1:18 PM, Guillaume Lelarge
> <guillaume@lelarge.info> wrote:
>> Quan Zongliang a écrit :
>>>[...]
>>> Add a new option like "Don't check database version."?
>>>
>> I don't think so. pgAdmin should display the warning box only if they
>> try to backup/restore, and not at startup time.
>
> This is to prevent the user seeing errors when they run an old pgAdmin
> against a newer server - it's not really about backup/restore (the
> thread kinda changed subject).
>

Oops, sorry about this...


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