Re: incorrect information in documentation

From: "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com>
To: ivanmulhin(at)gmail(dot)com, Pg Docs <pgsql-docs(at)lists(dot)postgresql(dot)org>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Subject: Re: incorrect information in documentation
Date: 2021-08-09 16:20:53
Message-ID: CAKFQuwYQacXmG9zMcOgbBtGp-1QusGjXQBfMez9GcHpv=AX6SQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-docs

On Mon, Aug 9, 2021 at 9:06 AM David G. Johnston <david(dot)g(dot)johnston(at)gmail(dot)com>
wrote:

> On Mon, Aug 9, 2021 at 8:02 AM PG Doc comments form <
> noreply(at)postgresql(dot)org> wrote:
>
>> Hello, on page
>> https://www.postgresql.org/docs/current/row-estimation-examples.html -
>> there
>> is a example:
>> selectivity = (1 - null_frac1) * (1 - null_frac2) * min(1/num_distinct1,
>> 1/num_distinct2)
>> = (1 - 0) * (1 - 0) / max(10000, 10000)
>> = 0.0001
>> in the first string " * min" and in the second " / max"
>> as I understand it isn't correct.
>>
>
> Division is just multiplication by the reciprocal so while the
> presentation here is inconsistent it is correct. Likewise, the larger a
> number the smaller its reciprocal, so the change from min to max also
> works.
>
>
FWIW this used to be presented with the calculation and formula in sync,
but the original had a simple typo in the calculation. When the typo got
fixed back in December of 2007 [1] the author of the patch simplified the
calculation at the same time. I suggest we update the formula line to
match the calculation presented.

David J.

1.
https://github.com/postgres/postgres/commit/f5678e8e07563e34ae4dc832546977d13edcd665

In response to

Responses

Browse pgsql-docs by date

  From Date Subject
Next Message Bruce Momjian 2021-08-09 18:05:50 Re: incorrect information in documentation
Previous Message David G. Johnston 2021-08-09 16:06:16 Re: incorrect information in documentation