Re: [pgAdmin4][RM#3140] Add service parameter - Mailing list pgadmin-hackers

From Murtuza Zabuawala
Subject Re: [pgAdmin4][RM#3140] Add service parameter
Date
Msg-id CAKKotZSt4is3xjhRbroRkuzBvGtbruGERQ2EX4dTTSvDX+3KnQ@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin4][RM#3140] Add service parameter  (Dave Page <dpage@pgadmin.org>)
Responses Re: [pgAdmin4][RM#3140] Add service parameter  (Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com>)
List pgadmin-hackers
Hi Dave,

I'll change the name and send you updated patch.


On Fri, Mar 9, 2018 at 9:25 PM, Dave Page <dpage@pgadmin.org> wrote:
HI

On Fri, Mar 9, 2018 at 11:47 AM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi,

PFA patch to add service parameter in server dialog.
- Docs updated
- Test case added for Service ID parameter

Please note,
I have extracted Connection class and Server manager class from our own custom Psycopg2 driver module.

Patch also covers RM#3120

 This patch seems a little confused. The "Service" and "Service ID" fields from pgAdmin 3 are very different things. The Redmine ticket seems to be asking for the Service field (the pg_service.conf service name), *not* Service ID (the operating system's service ID, used to start/stop the database server service).

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

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

pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: Re: [pgAdmin4][RM#3140] Add service parameter
Next
From: Dave Page
Date:
Subject: Re: pgAdmin 4 commit: Support EXPLAIN on Greenplum. Fixes #3097