From: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: Object translation mechanism is, umm, broken. |
Date: | 2011-06-11 12:37:41 |
Message-ID: | 1307795861.25444.0.camel@laptop |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On Sat, 2011-06-11 at 11:22 +0200, Guillaume Lelarge wrote:
> On Fri, 2011-06-03 at 14:24 +0100, Dave Page wrote:
> > On Fri, Jun 3, 2011 at 1:45 PM, Guillaume Lelarge
> > <guillaume(at)lelarge(dot)info> wrote:
> > >
> > > Hi,
> > >
> > > On Thu, 2011-06-02 at 08:54 +0000, Dave Page wrote:
> > > > [...]
> > > > This commit: http://git.postgresql.org/gitweb?p=pgadmin3.git;a=commitdiff;h=4b8ae9d82c3b879eb02dda60908ed9590a674ec4
> > > > (particularly the code in pgObject.cpp) is causing problems. I keep
> > > > running into objects for which we now get messages like "Refreshing
> > > > unknown object of type Replication", which I believe used to work just
> > > > fine. I've just hit it with the Slony replication cluster object, and
> > > > now have a sneaking suspicion it's also going to affect all the other
> > > > slony object types.
> > > >
> > >
> > > Right.
> > >
> > > > Aside from the fact that the code is incomplete, it's a serious
> > > > modularity violation. We shouldn't have a giant conditional statement
> > > > in a parent class that has knowledge of each of the different classes
> > > > that might be derived from that object. Instead, each of the child
> > > > classes should override a common function.
> > > >
> > > > Can you look at this please?
> > > >
> > >
> > > Working on it right now.
> > >
> >
> > Thanks - I appreciate it.
> >
>
> So I finally have something. Can you get a quick look at it to tell me
> if that's what you were looking for? I guess you'll be particularly
> interested in pgadmin/schema/pgObject.cpp.
>
> Thanks.
>
> BTW, still two issues on this (big) patch:
> * Greenplum objects are taken care of, but I have no way to check them
> * EDB objects still have no support. I can add that, but can't check
> the code
>
Same patch with EDB objects support (but still no tests on them). And
this time, compressed.
--
Guillaume
http://blog.guillaume.lelarge.info
http://www.dalibo.com
Attachment | Content-Type | Size |
---|---|---|
0001-Fix-our-i18n-support.patch.bz2 | application/x-bzip | 11.6 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Erwin Brandstetter | 2011-06-11 14:02:16 | Re: Database bar |
Previous Message | Guillaume Lelarge | 2011-06-11 09:22:04 | Re: Object translation mechanism is, umm, broken. |