Обсуждение: BUG #15629: Typo in Documentation

The following bug has been logged on the website:

Bug reference:      15629
Logged by:          Shouyu Luo
Email address:      syluo1990@hotmail.com
PostgreSQL version: 11.0
Operating system:   Ubuntu
Description:

https://www.postgresql.org/docs/current/ddl-partitioning.html

5.10.5. Partitioning and Constraint Exclusion
Constraint exclusion is only applied during query planning; unlike partition
pruning, it cannot be applied during query execution.

Is it supposed to be "unlike partition pruning, it can be applied during
query execution"?

On Mon, 11 Feb 2019 at 20:49, PG Bug reporting form
<noreply@postgresql.org> wrote:
> https://www.postgresql.org/docs/current/ddl-partitioning.html
>
> 5.10.5. Partitioning and Constraint Exclusion
> Constraint exclusion is only applied during query planning; unlike partition
> pruning, it cannot be applied during query execution.
>
> Is it supposed to be "unlike partition pruning, it can be applied during
> query execution"?

That's a bit confusing. "it" looks like must have been intended to
reference constraint exclusion, but since partition pruning was
mentioned afterwards, then it makes more sense to apply it to that.

Maybe it would be more clear to write:

Constraint exclusion is only applied during query planning; unlike
partition pruning which can also be applied during query execution.

Small patch doing it that way is attached.

-- 
 David Rowley                   http://www.2ndQuadrant.com/
 PostgreSQL Development, 24x7 Support, Training & Services

On Mon, Feb 11, 2019 at 5:25 PM David Rowley
<david.rowley@2ndquadrant.com> wrote:
> On Mon, 11 Feb 2019 at 20:49, PG Bug reporting form
> <noreply@postgresql.org> wrote:
> > https://www.postgresql.org/docs/current/ddl-partitioning.html
> >
> > 5.10.5. Partitioning and Constraint Exclusion
> > Constraint exclusion is only applied during query planning; unlike partition
> > pruning, it cannot be applied during query execution.
> >
> > Is it supposed to be "unlike partition pruning, it can be applied during
> > query execution"?
>
> That's a bit confusing. "it" looks like must have been intended to
> reference constraint exclusion, but since partition pruning was
> mentioned afterwards, then it makes more sense to apply it to that.
>
> Maybe it would be more clear to write:
>
> Constraint exclusion is only applied during query planning; unlike
> partition pruning which can also be applied during query execution.
>
> Small patch doing it that way is attached.

+1

Maybe, the semicolon should be replaced by a comma?

Thanks,
Amit

On 02/11/19 08:18, Amit Langote wrote:
> Maybe, the semicolon should be replaced by a comma?

I agree, and here is a version of the patch with that change.

Regards,
-Chap

On Tue, 12 Feb 2019 at 14:21, Chapman Flack <chap@anastigmatix.net> wrote:
>
> On 02/11/19 08:18, Amit Langote wrote:
> > Maybe, the semicolon should be replaced by a comma?
>
> I agree, and here is a version of the patch with that change.

Agreed about the comma.  Thanks for updating the patch.


-- 
 David Rowley                   http://www.2ndQuadrant.com/
 PostgreSQL Development, 24x7 Support, Training & Services

On Tue, Feb 12, 2019 at 03:09:30PM +1300, David Rowley wrote:
> Agreed about the comma.  Thanks for updating the patch.

Cool for me, so pushed down to v11.
--
Michael

On Tue, 12 Feb 2019 at 16:04, Michael Paquier <michael@paquier.xyz> wrote:
> Cool for me, so pushed down to v11.

Thanks for pushing.

(belated due to being on leave)

-- 
 David Rowley                   http://www.2ndQuadrant.com/
 PostgreSQL Development, 24x7 Support, Training & Services

On Mon, Feb 18, 2019 at 01:04:22PM +1300, David Rowley wrote:
> Thanks for pushing.
>
> (belated due to being on leave)

No problem.  Hope you had a good rest.
--
Michael