Skip site navigation (1) Skip section navigation (2)

Re: pgAdmin for Greenplum database?

From: Chuck McDevitt <cmcdevitt(at)greenplum(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: pgAdmin for Greenplum database?
Date: 2009-02-19 21:36:05
Message-ID: 2106D8DC89010842BABA5CD03FEA40617911A54F@EXVMBX018-10.exch018.msoutlookonline.net (view raw or flat)
Thread:
Lists: pgadmin-hackerspgadmin-support
The .xrc and xpm files 

> -----Original Message-----
> From: Dave Page [mailto:dpage(at)pgadmin(dot)org]
> Sent: Wednesday, February 11, 2009 2:44 AM
> To: Chuck McDevitt
> Cc: pgadmin-hackers
> Subject: Re: pgAdmin for Greenplum database?
> 
> On Tue, Feb 10, 2009 at 8:58 PM, Chuck McDevitt
> <cmcdevitt(at)greenplum(dot)com> wrote:
> >
> >> Can you post your patches ASAP so we can see how large they are and
> >> how much work will be involved?
> >>
> > And here are some of the new source files for the change.
> >
> > The changes aren't quite ready yet, I still have work to do.
> 
> Hi,
> 
> I've quickly eyeballed the patch and the additional files and don't
> see anything that gives me any cause for concern. Obviously I cannot
> test the Greenplum-specific code at the moment (I guess I'll have to
> download the trial at some point - yet another test VM :-p ). I did
> spot the following (minor) issues, at least some of which I would
> assume you're already aware of:
> 
> - The additional constants in keywords.h should have a
> product-specific suffix, ie. _GP
> 
> - The standard indent in the pgAdmin code is four spaces, not a tab.
> 
> - The online help is using the PostgreSQL help site. Do you have a
> version of the PG docs with the Greenplum additions that still uses
> the same file names? If so, there should be an appropriate help path
> option on frmOptions, if not, we need to figure out an acceptable
> alternative.
> 
> - The patch is missing images, headers, dialogue code (I assume there
> should be dlgResQueue.cpp and dlgPartition.cpp) and dialogue
> resources.
> 
> - precomp.h needs an update to include any new headers.
> 
> - Presumably Greenplum has it's own versions of pg_dump and friends,
> in which case we need a GP Bin Path option on frmOptions, and changes
> to frmBackup::getCmdPart1() and similar functions to call the correct
> binary.
> 
> Please be aware that we're expecting to go to beta within a few weeks
> (at the same time as PostgreSQL 8.4), so we'll want to get these
> patches in fairly soon.
> 
> --
> Dave Page
> EnterpriseDB UK:   http://www.enterprisedb.com

Attachment: dlgExtTable.xrc
Description: application/octet-stream (6.4 KB)
Attachment: exttable-sm.xpm
Description: image/x-xpixmap (1.7 KB)
Attachment: exttables.xpm
Description: image/x-xpixmap (2.1 KB)
Attachment: exttable.xpm
Description: image/x-xpixmap (2.6 KB)
Attachment: ex_redistribute_motion.xpm
Description: image/x-xpixmap (2.7 KB)
Attachment: ex_gather_motion.xpm
Description: image/x-xpixmap (2.7 KB)
Attachment: ex_broadcast_motion.xpm
Description: image/x-xpixmap (2.8 KB)

In response to

pgadmin-hackers by date

Next:From: Quan ZongliangDate: 2009-02-20 03:03:00
Subject: Re: About wxODBC
Previous:From: Chuck McDevittDate: 2009-02-19 21:33:43
Subject: Re: pgAdmin for Greenplum database?

pgadmin-support by date

Next:From: Dave PageDate: 2009-02-20 11:10:50
Subject: Re: auto fk index doesn't work when creating new table
Previous:From: Chuck McDevittDate: 2009-02-19 21:33:43
Subject: Re: pgAdmin for Greenplum database?

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group