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
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 |