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

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

Robert Haas <robertmhaas(at)gmail(dot)com> writes:
> On Tue, Oct 14, 2014 at 3:03 AM, David Rowley <dgrowleyml(at)gmail(dot)com> wrote:
>> Hmm, was my case above not compelling enough?

> Apparently not to Tom, but it made sense to me.

No, it wasn't. I'm not convinced either that that patch will get in at
all, or that it has to have regression tests of that particular form,
or that such a switch would be sufficient to make such tests platform
independent.

> I think we should
> find a way to do something about this - maybe making TIMING OFF also
> suppress that info is the simplest approach.

We intentionally did *not* make TIMING OFF do that to begin with, and
I think changing that behavior now has even less chance of escaping
push-back than the "planning time" change did.

If we're convinced we must do something, I think exposing the SUMMARY
ON/OFF flag (possibly after bikeshedding the name) that I implemented
internally yesterday would be the thing to do. But as I said, I find
the use-case for this pretty darn weak.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Amit Kapila 2014-10-16 14:09:22 Re: TODO : Allow parallel cores to be used by vacuumdb [ WIP ]
Previous Message Stephen Frost 2014-10-16 14:02:38 Re: Additional role attributes && superuser review