Re: Bug in the graphical explain

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>
Cc: Guillaume Lelarge <guillaume(at)lelarge(dot)info>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Bug in the graphical explain
Date: 2013-08-20 07:45:16
Message-ID: CA+OCxoz5_mQ5WoLTtceartx=FEkU_7ToXRNzfqZnhqNgwQbPGA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Works nicely - not entirely sure I understand how, but it does the job!!

Nice work - committed.

Thanks!

On Tue, Aug 20, 2013 at 8:15 AM, Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com
> wrote:

> On Fri, Jul 19, 2013 at 1:58 AM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> Ashesh; do we have any capacity in the team to spend a little time on
>> this at the moment? It'll affect the PEM SQL Profiler as well of
>> course...
>>
> First of all apology for very late reply.
> Here is the patch which resolves this problem.
>
>>
>> On Thu, Jul 18, 2013 at 9:26 PM, Guillaume Lelarge
>> <guillaume(at)lelarge(dot)info> wrote:
>> > Hi,
>> >
>> > I have no idea on how to fix this. I've attached the test case to
>> > reproduce it.
>> >
>> > First execute the test SQL file, then select the last UPDATE query, and
>> > hit F7. The graphical explain is kinda ugly because two nodes are one on
>> > the other. That's not the first time I meet this bug. For example, when
>> > you explain an DELETE query on a table with foreign keys.
>> >
>> > If you know how to fix this, that would be great because I'm completely
>> > lost here.
>> >
>> >
>> > --
>> > Guillaume
>> > http://blog.guillaume.lelarge.info
>> > http://www.dalibo.com
>> >
>> >
>> > --
>> > Sent via pgadmin-hackers mailing list (pgadmin-hackers(at)postgresql(dot)org)
>> > To make changes to your subscription:
>> > http://www.postgresql.org/mailpref/pgadmin-hackers
>> >
>>
>>
>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>
> --
>
> Thanks & Regards,
>
> Ashesh Vashi
> EnterpriseDB INDIA: Enterprise PostgreSQL Company<http://www.enterprisedb.com>
>
>
>
> *http://www.linkedin.com/in/asheshvashi*<http://www.linkedin.com/in/asheshvashi>
>

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2013-08-20 07:49:56 pgAdmin III commit: Tidy up the DROP/REASSIGN dialogue.
Previous Message Dave Page 2013-08-20 07:44:35 pgAdmin III commit: Fix a layout bug in the graphical explain renderer