Re: Hide 'Execution time' in EXPLAIN (COSTS OFF)

From: David Rowley <dgrowleyml(at)gmail(dot)com>
To: Christoph Berg <cb(at)df7cb(dot)de>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Robert Haas <robertmhaas(at)gmail(dot)com>, Andres Freund <andres(at)2ndquadrant(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>, ronan(at)dunklau(dot)fr, Markus Wanner <markus(at)bluegap(dot)ch>
Subject: Re: Hide 'Execution time' in EXPLAIN (COSTS OFF)
Date: 2014-10-12 10:13:27
Message-ID: CAApHDvoVzBTzLJbD9VfaznWo6jooK1k6-7rFQ8zYM9H7ndCcSA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Sep 24, 2014 at 8:02 AM, Christoph Berg <cb(at)df7cb(dot)de> wrote:

> Re: Tom Lane 2014-09-23 <15155(dot)1411493559(at)sss(dot)pgh(dot)pa(dot)us>
> > Robert Haas <robertmhaas(at)gmail(dot)com> writes:
> > > On Sat, Sep 20, 2014 at 4:13 PM, Christoph Berg <cb(at)df7cb(dot)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;
QUERY PLAN
---------------------------------------------------
Hash Join (actual rows=1 loops=1)
Hash Cond: (b.c_id = c.id)
-> Hash Join (actual rows=1 loops=1)
Hash Cond: (a.b_id = b.id)
-> Seq Scan on a (actual rows=1 loops=1)
-> Hash (never executed)
-> Seq Scan on b (never executed)
-> Hash (never executed)
-> Seq Scan on c (never executed)
Execution time: 0.092 ms
(10 rows)

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.

Regards

David Rowley

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2014-10-12 11:17:00 Re: Hide 'Execution time' in EXPLAIN (COSTS OFF)
Previous Message Andres Freund 2014-10-12 09:36:53 Re: [PATCH] PostgreSQL 9.4 mmap(2) performance regression on FreeBSD...