Re: [HACKERS] Reporting planning time with EXPLAIN - Mailing list pgsql-hackers

From Ashutosh Bapat
Subject Re: [HACKERS] Reporting planning time with EXPLAIN
Date
Msg-id CAFjFpRfbwONMqKYfPAuLVsG4YpjbR=BD_ToyKoLUGNDgxm-KJA@mail.gmail.com
Whole thread Raw
In response to Re: [HACKERS] Reporting planning time with EXPLAIN  (Stephen Frost <sfrost@snowman.net>)
List pgsql-hackers
On Thu, Mar 9, 2017 at 2:58 AM, Stephen Frost <sfrost@snowman.net> wrote:
> Ashutosh,
>
> * Ashutosh Bapat (ashutosh.bapat@enterprisedb.com) wrote:
>> On Tue, Mar 7, 2017 at 9:23 PM, Stephen Frost <sfrost@snowman.net> wrote:
>> > I'm going through these with an eye towards committing them soon.  I've
>> > already adjusted some of the documentation and comments per our earlier
>> > discussion
>>
>> Thanks a lot.
>
> I've pushed this with the editorialization of the documentation which
> we discussed up-thread along with some improvements to the comments
> along with your latest variable name suggestions and changes as
> discussed.

The documentation looks pretty good and comment changes look good too.
The way those two have been phrased, it's easy to understand the
defaults for SUMMARY, which otherwise could have been confusing. The
test looks good too.

Thanks for all the improvements.

-- 
Best Wishes,
Ashutosh Bapat
EnterpriseDB Corporation
The Postgres Database Company



pgsql-hackers by date:

Previous
From: Ashutosh Bapat
Date:
Subject: Re: [HACKERS] PATCH: psql show index with type info
Next
From: Ashutosh Bapat
Date:
Subject: Re: [HACKERS] foreign partition DDL regression tests