Re: [COMMITTERS] pgsql: Include planning time in EXPLAIN ANALYZE output. - Mailing list pgsql-hackers

From Robert Haas
Subject Re: [COMMITTERS] pgsql: Include planning time in EXPLAIN ANALYZE output.
Date
Msg-id CA+TgmoY85Kdps6f9W_n105Xpq0apnpJ-=_jMtPBKTRNwTwiORg@mail.gmail.com
Whole thread Raw
In response to Re: [COMMITTERS] pgsql: Include planning time in EXPLAIN ANALYZE output.  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: [COMMITTERS] pgsql: Include planning time in EXPLAIN ANALYZE output.
List pgsql-hackers
On Sun, Feb 2, 2014 at 11:13 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Peter Geoghegan <pg@heroku.com> writes:
>> On Wed, Jan 29, 2014 at 1:10 PM, Robert Haas <rhaas@postgresql.org> wrote:
>>> Include planning time in EXPLAIN ANALYZE output.
>
>> Isn't it perhaps a little confusing that "Planning time" may well
>> exceed "Total runtime"?
>
> Perhaps s/Total runtime/Execution time/ ?

I'm not really feeling a compelling need to change that.  We've been
displaying total runtime - described exactly that way - for many
releases and it's surely is confusing to the novice that the time
reported can be much less than the time reported by psql's \timing
option, usually because of planning time.  But adding the planning
time to the output seems to me to make that better, not worse.  If the
user can't figure out that runtime != planning time, I'm not sure
they'll be able to figure out execution time != planning time, either.

One of the reasons it's called "Total runtime", or so I've always
assumed, is because it's more inclusive than the time shown for the
root node of the plan tree.  Specifically, it includes the time
required to fire triggers.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



pgsql-hackers by date:

Previous
From: Christian Kruse
Date:
Subject: Re: Patch: Show process IDs of processes holding a lock; show relation and tuple infos of a lock to acquire
Next
From: "Tomas Vondra"
Date:
Subject: Re: GIN improvements part2: fast scan