Re: [pgAdmin][RM4335] Add EXPLAIN option SETTINGS and SUMMARY

From: Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin][RM4335] Add EXPLAIN option SETTINGS and SUMMARY
Date: 2019-06-27 04:31:24
Message-ID: CAM9w-_=D2m7n7SAMLL2ecViRLJGiONUZeEtLhDA01kA7v7QPGQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi,

On Wed, Jun 26, 2019 at 8:22 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:

> Hi
>
> Something isn't working correctly - I turned on the summary option, and
> this is the query that was run when I hit the EXPLAIN button:
>
> EXPLAIN (FORMAT JSON,ANALYZE False,VERBOSE False,COSTS False,TIMING
> False,SUMMARY False,BUFFERS False) select * from pg_tables
>

> Aside from the formatting (s/True/true, s/False/false, missing spaces
> etc), shouldn't it say SUMMARY true? I've cleared the browser and bundle
> caches, rebundled etc.
>
I think I misunderstood the documents -
https://www.postgresql.org/docs/12/sql-explain.html. Few of the parameters
are applicable only to EXPLAIN ANALYSE and not to EXPLAIN. Will make the
changes, along with formatting to existing codes as well.

>
> Shouldn't there also be documentation updates?
>
Yeah. (I always miss that :/)

>
> On Wed, Jun 26, 2019 at 7:59 AM Aditya Toshniwal <
> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>
>> Hi Hackers,
>>
>> Attached is the patch to add support for SETTINGS(v12+) and SUMMARY(v10+)
>> explain options. These options will be visible only if version is supported.
>>
>> I have also created and fixed RM4395. The explain options changed on one
>> query tool should not change explain options on other query tools or
>> preferences. The initial values will be taken from preferences are usual.
>>
>> Kindly review.
>>
>> --
>> Thanks and Regards,
>> Aditya Toshniwal
>> Software Engineer | EnterpriseDB India | Pune
>> "Don't Complain about Heat, Plant a TREE"
>>
>
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>

--
Thanks and Regards,
Aditya Toshniwal
Software Engineer | EnterpriseDB India | Pune
"Don't Complain about Heat, Plant a TREE"

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Usman Muzaffar 2019-06-27 08:18:29 feature tests fix
Previous Message Khushboo Vashi 2019-06-27 04:00:56 Re: Getting Involved