Re: [pgadmin4][Patch] Greenplum specific DDL and Dashboard display

. >> >> -- Thanks, Ashesh >> >>> >>> Dave Cramer >>> >>> On 24 August 2017 at 23:00, Teng Zhang wrote: >>> >>>> Sure, you can get as much as you like. >>>> Thanks >>>> >>>> -

Re: [pgadmin4][Patch] Greenplum specific DDL and Dashboard display

ge -- >>> From: Dave Cramer >>> Date: Thu, Aug 24, 2017 at 8:34 PM >>> Subject: Re: [pgadmin4][Patch] Greenplum specific DDL and Dashboard >>> display >>> To: Teng Zhang >>> Cc: Ashesh Vashi , pgadmin-hackers < >>> pgadmin

Re: [pgadmin4][Patch] Greenplum specific DDL and Dashboard display

> >> Sure, you can get as much as you like. >> Thanks >> >> -- Forwarded message -- >> From: Dave Cramer >> Date: Thu, Aug 24, 2017 at 8:34 PM >> Subject: Re: [pgadmin4][Patch] Greenplum specific DDL and Dashboard >> display

Re: [pgadmin4][Patch] Greenplum specific DDL and Dashboard display

I'm suggesting that it be pushed Dave Cramer On 24 August 2017 at 23:00, Teng Zhang wrote: > Sure, you can get as much as you like. > Thanks > > -- Forwarded message -- > From: Dave Cramer > Date: Thu, Aug 24, 2017 at 8:34 PM > Subject: Re: [pgadmin4]

Fwd: [pgadmin4][Patch] Greenplum specific DDL and Dashboard display

Sure, you can get as much as you like. Thanks -- Forwarded message -- From: Dave Cramer Date: Thu, Aug 24, 2017 at 8:34 PM Subject: Re: [pgadmin4][Patch] Greenplum specific DDL and Dashboard display To: Teng Zhang Cc: Ashesh Vashi , pgadmin-hackers < pgadmin-h

Re: [pgadmin4][Patch] Greenplum specific DDL and Dashboard display

Can we get some movement on this patch? This seems like a more sane way to go to support different "products" Dave Cramer On 22 August 2017 at 16:56, Dave Cramer wrote: > > On 22 August 2017 at 16:41, Dave Cramer wrote: > >> Ok, >> >> Surely this : >> >> self.table_template_path = 'table/sql/'

Re: [pgadmin4][Patch] Greenplum specific DDL and Dashboard display

On 22 August 2017 at 16:41, Dave Cramer wrote: > Ok, > > Surely this : > > self.table_template_path = 'table/sql/' + ( > +'#{0}#{1}#'.format(server_type, ver) > +if server_type == 'gpdb' else > +'#{0}#'.format(ver) > +) > > could be writ

Re: [pgadmin4][Patch] Greenplum specific DDL and Dashboard display

Ok, Surely this : self.table_template_path = 'table/sql/' + ( +'#{0}#{1}#'.format(server_type, ver) +if server_type == 'gpdb' else +'#{0}#'.format(ver) +) could be written in a more readable manner ?? Dave Cramer On 22 August 2017 at

Re: [pgadmin4][Patch] Greenplum specific DDL and Dashboard display

Hi, I've been able to get back to this and test it. So far so good. It applies more or less cleanly against 1.6 and everything I've tried so far works I'll update more as I test it. Thanks Dave Cramer On 21 August 2017 at 05:29, Teng Zhang wrote: > Hi, > > Thanks for the review, here is a fi

Re: [pgadmin4][Patch] Greenplum specific DDL and Dashboard display

Hi, Thanks for the review, here is a fixed patch working for GBDP which shows the appropriate graphs. In this fix, we toke out the changes to diver/psycopg2 and implemented the greenplum version checking process in the ppas way mentioned by Dave Cramer. Regards, Teng Zhang & Hao Wang On Mon, Aug

Re: [pgadmin4][Patch] Greenplum specific DDL and Dashboard display

On Mon, Aug 21, 2017 at 1:23 PM, Dave Page wrote: > Ashesh, do you have a recommended way to do this? > > I haven't looked at the patch, but I assume it adds a database driver > module for GPDB? > I have not looked at the patch yet. I will take a look at it. -- Thanks & Regards, Ashesh Vashi E

Re: [pgadmin4][Patch] Greenplum specific DDL and Dashboard display

Ashesh, do you have a recommended way to do this? I haven't looked at the patch, but I assume it adds a database driver module for GPDB? On Mon, Aug 21, 2017 at 8:50 AM, Jing Li wrote: > Hi Dave, > > Since we're hoping to get this change working for GPDB we've currently > using this method to d

Re: [pgadmin4][Patch] Greenplum specific DDL and Dashboard display

Hi Dave, Since we're hoping to get this change working for GPDB we've currently using this method to detect if it's gpdb and show the appropriate graphs. Right now it displays errors on the dashboard if it's connected to a gpdb server.For this patch specifically, the goal is to improve the experie

Re: [pgadmin4][Patch] Greenplum specific DDL and Dashboard display

Hi Violet. I don't really like the way this has been implemented. It adds a variable which is only used for gpdb. There are other places in the code where the behaviour is changed if the server is ppas or regular postgres. Candidly I think all of this needs restructuring. Dave Cramer On 15 Aug

Re: [pgadmin4][Patch] Greenplum specific DDL and Dashboard display

Hi, Any comment on this patch? If no, will it be committed soon? Thanks, Violet On Wed, Aug 9, 2017 at 12:05 PM, Sarah McAlear wrote: > Hi Hackers! > > This patch enables Greenplum users to see the same charts on the dashboard > as postgres users. It also adds some additional information to th

[pgadmin4][Patch] Greenplum specific DDL and Dashboard display

Hi Hackers! This patch enables Greenplum users to see the same charts on the dashboard as postgres users. It also adds some additional information to the DDL that is Greenplum specific and necessary to create a new table. Thanks! Sarah diff --git a/web/pgadmin/browser/server_groups/servers/datab