Re: [PG13] Planning (time + buffers) data structure in explain plan (format text) - Mailing list pgsql-hackers

From Fujii Masao
Subject Re: [PG13] Planning (time + buffers) data structure in explain plan (format text)
Date
Msg-id 0d074a53-fd91-b1dd-a529-173d07627465@oss.nttdata.com
Whole thread Raw
In response to Re: [PG13] Planning (time + buffers) data structure in explain plan (format text)  (Julien Rouhaud <rjuju123@gmail.com>)
Responses Re: [PG13] Planning (time + buffers) data structure in explain plan (format text)  (Pierre Giraud <pierre.giraud@dalibo.com>)
List pgsql-hackers

On 2020/08/20 22:34, Julien Rouhaud wrote:
> On Thu, Aug 20, 2020 at 12:29 PM David Rowley <dgrowleyml@gmail.com> wrote:
>>
>> On Thu, 20 Aug 2020 at 19:58, Fujii Masao <masao.fujii@oss.nttdata.com> wrote:
>>>
>>> On 2020/08/20 13:00, David Rowley wrote:
>>>> I forgot to mention earlier, you'll also need to remove the part in
>>>> the docs that mentions BUFFERS requires ANALYZE.
>>>
>>> Thanks for the review! I removed that.
>>> Attached is the updated version of the patch.
>>> I also added the regression test performing "explain (buffers on)"
>>> without "analyze" option.
>>
>> Looks good to me.
> 
> Looks good to me too.

David and Julien, thanks for the review! I'd like to wait for
Pierre's opinion about this change before committing the patch.

Pierre,
could you share your opinion about this change?

Regards,

-- 
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION



pgsql-hackers by date:

Previous
From: Fujii Masao
Date:
Subject: Re: Transactions involving multiple postgres foreign servers, take 2
Next
From: Tom Lane
Date:
Subject: ReplicationSlotsComputeRequiredXmin seems pretty questionable