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.
/Andreas