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

From: Surinder Kumar <surinder(dot)kumar(at)enterprisedb(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4][Patch]: RM1569 - Right-clicking an disconnected EPAS server shows "Create Resource Group" option
Date: 2016-08-19 21:14:54
Message-ID: CAM5-9D8_cHn8cV29-tRWBsitOUfZc7YZurfGDVg8LC=hzZzzrw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Sat, Aug 20, 2016 at 1:39 AM, Dave Page <dpage(at)pgadmin(dot)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(dot)kumar(at)enterprisedb(dot)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(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgadmin-hackers
>
>

Attachment Content-Type Size
RM1569_v1.patch application/octet-stream 1.9 KB

In response to

Responses

Browse pgadmin-hackers by date

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