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 |
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 |