Re: Planning time in explain/explain analyze

From: Andreas Karlsson <andreas(at)proxel(dot)se>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Greg Stark <stark(at)mit(dot)edu>, Stephen Frost <sfrost(at)snowman(dot)net>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Planning time in explain/explain analyze
Date: 2014-01-29 20:13:28
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

On 01/29/2014 09:01 PM, Robert Haas wrote:
> Cool. I propose adding one parameter rather than two to
> ExplainOnePlan() and making it of type instr_time * rather than
> passing an instr_time and a bool. If you don't want to include the
> planning time, pass NULL; if you do, pass a pointer to the instr_time
> you want to print. I think that would come out slightly neater than
> what you have here.

Excellent suggestion, thanks! New patch attached.


Attachment Content-Type Size
explainplantime-v6.diff text/x-patch 15.3 KB

In response to


Browse pgsql-hackers by date

  From Date Subject
Next Message Heikki Linnakangas 2014-01-29 20:17:22 Re: [PATCH] Use MAP_HUGETLB where supported (v3)
Previous Message Simon Riggs 2014-01-29 20:11:38 Re: Patch: show xid and xmin in pg_stat_activity and pg_stat_replication