On Wed, Sep 24, 2014 at 8:02 AM, Christoph Berg <cb@df7cb.de> wrote:
Re: Tom Lane 2014-09-23 <15155.1411493559@sss.pgh.pa.us> > Robert Haas <robertmhaas@gmail.com> writes: > > On Sat, Sep 20, 2014 at 4:13 PM, Christoph Berg <cb@df7cb.de> wrote: > >> Can we have "EXPLAIN (timing off)" in 9.4+ hide the "Planning time" > >> line? That would even be backwards compatible with 9.x where it would > >> be a no-op. > > > I don't think that'll work becuase: > > > /* check that timing is used with EXPLAIN ANALYZE */ > > if (es.timing && !es.analyze) > > ereport(ERROR, > > (errcode(ERRCODE_INVALID_PARAMETER_VALUE), > > errmsg("EXPLAIN option TIMING > > requires ANALYZE"))); > > It looks to me like that would complain about EXPLAIN (TIMING ON), > not the case Christoph is suggesting. What he proposes seems a bit > odd and non-orthogonal, but we could make the code do it if we wanted.
I don't think this warrants a new flag, and TIMING OFF seems to be the right naming for it. (In fact it was the first I tried, and I was cursing quite a bit over the lack of configurability until I realized that COSTS OFF disabled the planning time display as well.) It might be a bit odd, but it's easy to remember.
I'm pretty interested in seeing something change around here.
The patch I'm working on at the moment (INNER JOIN removals) implements "skipping" of joins at execution time rather than planning time. Currently I'm working on the regression test for this and it's not all that easy due to the execution time appearing in the results.
An explain analyze output from master with the patch can look something like:
explain (analyze, costs off, timing off)
select a.* from a inner join b on a.b_id = b.id inner join c on b.c_id = c.id;
From this I can see easily that the joins to b and c were skipped, however the output the way it is at the moment is quite useless for regression testing with.