Re: [pgAdmin4][Patch]: RM1569 - Right-clicking an disconnected EPAS server shows "Create Resource Group" option - Mailing list pgadmin-hackers

From Surinder Kumar
Subject Re: [pgAdmin4][Patch]: RM1569 - Right-clicking an disconnected EPAS server shows "Create Resource Group" option
Date
Msg-id CAM5-9D8_cHn8cV29-tRWBsitOUfZc7YZurfGDVg8LC=hzZzzrw@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin4][Patch]: RM1569 - Right-clicking an disconnected EPAS server shows "Create Resource Group" option  (Dave Page <dpage@pgadmin.org>)
Responses Re: [pgAdmin4][Patch]: RM1569 - Right-clicking an disconnected EPAS server shows "Create Resource Group" option  (Dave Page <dpage@pgadmin.org>)
List pgadmin-hackers
On Sat, Aug 20, 2016 at 1:39 AM, Dave Page <dpage@pgadmin.org> wrote:
Hi

That's not a reliable check - you can just as easily use postgres as the maintenance db on epas.
​Yes, somehow i missed it.​

Why don't we just hide the option if disconnected, and if connected, then check the server type?
​The issue is that we are not getting 'server_type' in response after connect/disconnect of server. that's why checks gets failed.​ it always shows server type as 'pg'.
now it has been taken care in the updated patch.


-- 
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK:http://www.enterprisedb.com
The Enterprise PostgreSQL Company

On 19 Aug 2016, at 19:00, Surinder Kumar <surinder.kumar@enterprisedb.com> wrote:

Hi

Issue:
Two reasons:
1) In Resource group menu, the check for server connected or not is missing.
2) When server is disconnected, server type is always set to 'pg' which is wrong and thus function always returns true.

Solution:
If server is disconnected, we should check server type based on its maintenance db.
If maintenance db is 'edb' its ppas server, otherwise 'pg'.

Please review attached patch.

Thanks,
Surinder Kumar
<RM1569.patch>

--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers

Attachment

pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: Re: [pgAdmin4][Patch]: RM1569 - Right-clicking an disconnected EPAS server shows "Create Resource Group" option
Next
From: Ashesh Vashi
Date:
Subject: Re: PATCH/s: RM#1387 - Bad handling of missing connection database server