Re: [PATCH] Add EXPLAIN (ALL) shorthand - Mailing list pgsql-hackers

From Tom Lane
Subject Re: [PATCH] Add EXPLAIN (ALL) shorthand
Date
Msg-id 49766.1463695967@sss.pgh.pa.us
Whole thread Raw
In response to Re: [PATCH] Add EXPLAIN (ALL) shorthand  (Евгений Шишкин <itparanoia@gmail.com>)
Responses Re: [PATCH] Add EXPLAIN (ALL) shorthand  (Евгений Шишкин <itparanoia@gmail.com>)
List pgsql-hackers
Евгений Шишкин <itparanoia@gmail.com> writes:
>> On 19 May 2016, at 22:59, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>> I'm not sure this is well thought out.  It would mean for example that
>> we could never implement EXPLAIN options that are mutually exclusive
>> ... at least, not without having to redefine ALL as all-except-something.
>> Non-boolean options would be problematic as well.

> Maybe EVERYTHING would be ok.

That's not really getting at the substance of my complaint, which is that
I foresee regretting inventing such an option later, when we wish to invent
some option that it's not reasonable for ALL/EVERYTHING/WHATEVER to turn
"on" (if indeed the new option is "off"/"on" in the first place).  We'll
either have to accept an arbitrary/inconsistent definition of what ALL
does to that option, or have to explain that ALL doesn't mean all.

My own experience is that I seldom want ANALYZE and VERBOSE at the same
time, so I'm doubtful that I'd find this proposal helpful even just
considering the current option set.

I'm also not convinced about the semantics of, say, EXPLAIN (ALL, COSTS
OFF).  I see what the patch will do, but I'm not convinced I like that
(and I am convinced that the patch's documentation is misleading about
it).

I'm a bit inclined to think that what this is really about is that we
made the wrong call on the BUFFERS option, and that it should default
to ON just like COSTS and TIMING do.  Yeah, that would be an incompatible
change, but that's what major releases are for no?
        regards, tom lane



pgsql-hackers by date:

Previous
From: "David G. Johnston"
Date:
Subject: Re: [PATCH] Add EXPLAIN (ALL) shorthand
Next
From: Gavin Flower
Date:
Subject: Re: [PATCH] Add EXPLAIN (ALL) shorthand