Postgres Pro
Downloads
Home   >   mailing lists

Re: [pgadmin4][Patch] Greenplum specific DDL and Dashboard display - Mailing list pgadmin-hackers

From Jing Li
Subject Re: [pgadmin4][Patch] Greenplum specific DDL and Dashboard display
Date
Msg-id dddc8e28-03f4-b9c7-1052-7ce73aaaa9e5@mixmax.com
Whole thread Raw
In response to Re: [pgadmin4][Patch] Greenplum specific DDL and Dashboard display  (Dave Cramer <davecramer@gmail.com>)
Responses Re: [pgadmin4][Patch] Greenplum specific DDL and Dashboard display
List pgadmin-hackers
Tree view
a {word-wrap:normal;word-break:break-word;}.background-contain {background-size:contain;}@media only screen and (max-width:600px) {.container {-webkit-text-size-adjust:none !important;}.container,.palm-one-whole {width:100% !important;min-width:100% !important;}.palm-one-half {width:50% !important;min-width:50% !important;box-sizing:border-box;}blockquote .container,blockquote .container div,blockquote .container table {width:auto !important;min-width:0 !important;position:relative !important;}img {max-width:100%;}.border-outer,.border-middle,.border-inner,.inner,[title="separator"] {width:100% !important;}.innercell {padding:8px !important;}.palm-block {display:block !important;}td.palm-one-whole {display:inline-block !important;padding:0;}td.palm-one-whole:first-child:not(:only-child) {margin-bottom:16px;}td.hostname {padding-top:3px !important;}}@media only screen and (min-width:601px) {.preview-card {max-width:600px !important;}}@media only screen and (min-device-width :320px) and (max-device-width :568px),only screen and (min-device-width :768px) and (max-device-width :1024px),only screen and (max-device-width:640px),only screen and (max-device-width:667px),only screen and (max-width:480px){.container {width:100% !important;min-width:100% !important;}.p,.small,li,font[size="2"],font[size="3"] {font-size:1em !important;}}@media only screen and (min-device-width :320px) and (max-device-width :568px),only screen and (min-device-width :768px) and (max-device-width :1024px),only screen and (min-device-width :1224px) {.message-wrapper {padding-top:6px;}.apple-only[style] {display:block !important;max-height:none !important;line-height:normal !important;overflow:visible !important;height:auto !important;width:100% !important;position:relative !important;}.no-apple {display:none !important;}form {font-size:inherit;}input[type="text"] {height:43px;padding-left:4px !important;}button:hover {cursor:pointer;}}@media only screen and (min-device-width :1224px) {.apple-mail-form {display:block !important;background-color:white !important;}}* [office365] .outlook-com-hidden {display:none !important;}* [office365] .outlook-com-button {display:block;}* [office365] .outlook-com-only {display:block !important;max-height:none !important;line-height:normal !important;overflow:visible !important;height:auto !important;width:100% !important;position:relative !important;}.ExternalClass {width:100%;}.ExternalClass .outlook-com-button {display:block;}.ExternalClass button {height:auto;}.ExternalClass .outlook-com-hidden {display:none !important;}.ExternalClass .outlook-com-only {display:block !important;max-height:none !important;line-height:normal !important;overflow:visible !important;height:auto !important;width:100% !important;position:relative !important;}.ExternalClass .ecxlabels {display:none !important;}.ExternalClass .ecxlabels {display:none !important;}.ExternalClass .ecxarrow {display:none !important;}.ExternalClass cite >div + div {padding:0 0 4px 0;}.ExternalClass .h1 {padding-bottom:5px;}.ExternalClass .h2 {padding-bottom:5px;}.ExternalClass .h3 {padding-bottom:5px;}.ExternalClass [lang="brand-pinterest"] {width:280px !important;}
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 experience for greenplum users so they can get the same information as someone connected to a postgres server. 

I do agree that this is a bigger discussion about how we handle behavior change overall if it's regular postgres or something else. Let's talk about how we can restructure this behavior in a wider context. Are you open to meeting about it?

Thanks,
~Jing



On Fri, Aug 18, 2017 5:37 AM, Dave Cramer davecramer@gmail.com wrote:
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 August 2017 at 23:29, Violet Cheng <vcheng@pivotal.io> wrote:
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 <smcalear@pivotal.io> 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 the DDL that is Greenplum specific and necessary to create a new table.

Thanks!
Sarah




pgadmin-hackers by date:

Previous
From: Ashesh Vashi
Date:
Subject: Re: PATCH: pgagent connection string parsing
Next
From: Dave Page
Date:
Subject: Re: [pgadmin4][Patch] Greenplum specific DDL and Dashboard display