On Wed, Jan 29, 2014 at 3:13 PM, Andreas Karlsson <andr...@proxel.se> wrote:
> 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.

Committed with minor doc changes.

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


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to