From: | Magnus Hagander <magnus(at)hagander(dot)net> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: pgadmin crash on refresh |
Date: | 2011-01-21 18:36:12 |
Message-ID: | AANLkTi=qmNUBtQidkbveR8dUiYx3Y-hJ=uysNCjQmJHH@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On Fri, Jan 21, 2011 at 13:35, Dave Page <dpage(at)pgadmin(dot)org> wrote:
> On Fri, Jan 21, 2011 at 12:20 PM, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
>> On Wed, Jan 19, 2011 at 14:55, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>> On Wed, Jan 19, 2011 at 12:18 PM, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
>>>> Opened database->catalogs->pg_catalogs->tables. Right click + refresh = crash.
>>>>
>>>> Don't have time to dig in right now myself, but here's a backtrace if
>>>> somebody wants to get started, or perhaps has a clue what it could
>>>> be...
>>>>
>>>> this is with current git master tip, debugging enabled.
>>>
>>> Hmm, can't reproduce it on Windows. It does look an awful lot like
>>
>> Oh, additional point: you need to actually *expand* the tables node,
>> or it doesn't crash. Did you by any chance not do that?
>
> I did do that.
>
>>> this though: http://git.postgresql.org/gitweb?p=pgadmin3.git;a=commit;h=9ec6b29fe3fbd9cc196d5e91b55f3697f02bcad8
>>
>> Nope. I reverted that patch (well, due to the style patch i couldn't,
>> but i commented out that one line it seems to add) and it still
>> crashes for me.
>
> There's little chance that that patch would cause (or fix) the problem
> you see - but a similar patch in a different place might be the fix.
>
I've created ticket 299 for this for now, so we don't loose track of it.
--
Magnus Hagander
Me: http://www.hagander.net/
Work: http://www.redpill-linpro.com/
From | Date | Subject | |
---|---|---|---|
Next Message | Guillaume Lelarge | 2011-01-22 00:12:09 | Re: Support for pg_stat_database_conflicts and pg_stat_replication |
Previous Message | Magnus Hagander | 2011-01-21 18:05:44 | pgAdmin III commit: Add a guru hint explaining why VACUUM FULL is bad |