Thanks for reviewing again. On 9 May 2018 at 01:32, Justin Pryzby <pry...@telsasoft.com> wrote: > On Mon, May 07, 2018 at 06:00:59PM +1200, David Rowley wrote: >> Many thanks for reviewing this. > > 2nd round - from the minimalist department: > > + partitions which cannot possibly contain any matching records. > maybe: partitions which cannot match any records.
I don't think that's an improvement. I don't think there's such a thing as "partitions which match records". A partition can contain a record, it never matches one. > + <para> > + Partition pruning done during execution can be performed at any of the > + following times: > > remove "done"? Removed. > + number of partitions which were removed during this phase of pruning > by > remove "of prunning" Removed. v3 attached. -- David Rowley http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services
further_enable_partition_pruning_doc_updates_v3.patch
Description: Binary data