From: | Jeff Adams <jadams(at)azavea(dot)com> |
---|---|
To: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
Cc: | Michael Shapiro <mshapiro51(at)gmail(dot)com>, Peter Geoghegan <peter(dot)geoghegan86(at)gmail(dot)com>, pgadmin-support(at)postgresql(dot)org |
Subject: | Re: Bug with exceptionally long values. |
Date: | 2010-07-28 11:59:09 |
Message-ID: | AANLkTi=gMQQqfGF+QvYV6uEDaUT2ekmd18+ZPq+o15sj@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-support |
The class that I experience the problem with is the ctlSQLGrid, which
extends wxGrid. I don't know wxWidgets at all, but in the grid sample class
they have an example of setting a cell renderer, perhaps you could add a
cell renderer that changes the background color [or even renders a string
that says "Sorry, too long!"] based on the strlen? That should have fairly
minimal performance impact since it will only affect the N rows being
actually rendered, not the entire result set.
The example is /wxWidgets-2.8.11/samples/grid/griddemo.cpp line 299.
Jeff
On Tue, Jul 27, 2010 at 4:39 PM, Guillaume Lelarge
<guillaume(at)lelarge(dot)info>wrote:
> Le 27/07/2010 22:26, Jeff Adams a écrit :
> > The bug is definitely a wxWidgets bug, and exists both in 2.8.11 and
> 2.9.1.
> > I was able to reproduce it in their "grid" sample.
> >
> > I submitted a bug to the wxWidgets project:
> >
> > http://trac.wxwidgets.org/ticket/12276
> >
> > Do you think pgAdmin should implement some sort of temporary fix in the
> > meanwhile?
> >
>
> It would be best to have a (even temporary) fix. I'm not sure how we can
> do that. Having a special background colour could do that. But that
> means we need to check the length of every cells. I suppose that, if you
> have cells with more than 50k characters, you'll also have big tables.
> It sure will take a lot of time to check that. And what should we do if
> we find cells with more than X thousand characters? Highlight them? Draw
> something? I don't know. All I can say is that it won't be cheap
> (performance-wise).
>
>
> --
> Guillaume
> http://www.postgresql.fr
> http://dalibo.com
>
--
Jeff Adams
Azavea (formerly Avencia) | T: 215.701.7717 | F: 215.925.2663
www.azavea.com
From | Date | Subject | |
---|---|---|---|
Next Message | Guillaume Lelarge | 2010-07-28 13:19:02 | Re: Bug with exceptionally long values. |
Previous Message | Guillaume Lelarge | 2010-07-28 09:24:33 | Re: It takes a long time for pgAdmin to retrieve database schema details |