Re: Lightspeed for frmQuery and other issues.

[Andreas Pflug <pgadmin(at)pse-consulting(dot)de>]

Dave Page wrote:
> -----Original Message-----
> From: "Andreas Pflug"<pgadmin(at)pse-consulting(dot)de>
> Sent: 30/04/06 16:19:51
> To: "Dave Page"<dpage(at)vale-housing(dot)co(dot)uk>
> Cc: "pgadmin-hackers(at)postgresql(dot)org"<pgadmin-hackers(at)postgresql(dot)org>
> Subject: Re: Lightspeed for frmQuery and other issues.
>
> > It wasn't removed explicitely, but the underlying class that didn't meet 
> > the requirement was backed out. You've painted the wall before 
> > wallpapering it.
>
>
> Quickreport sat over that class as well - is that now broken too?

That's in frmQuery only, right? If so, the local solution is fine. If 
it's more global, it should go somehow to dlgClasses.
> Please restore the functionality or I will back out the patch until it is completed in it's entirety.

Which functionality?
> You complain about the work that led to significant speed increase from the original code, but at least we busted a gut to make sure it didn't break any existing functionality.

Again: I said early enough the base was wrong. And I didn't see *any* 
speed increase on linux. Probably, any virtual solution will be ok, but 
any non-virtual solution *cant* be ok.

> > It's not a problem of the factories, they do what they should and will 
> > work *if used*. 
>
>
> Like I said, I'll look at this. I didn't grok that the failed attempt you mentioned was only a minor rejig.

It should have been, but things are horribly intervowen. I'd have to 
understand in detail what's going on to continue, because apparently 
removing MNU_XXX entries would kill methods too.

Regards,
Andreas