From: | atorikoshi <torikoshi_atsushi_z2(at)lab(dot)ntt(dot)co(dot)jp> |
---|---|
To: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Fix a typo in README.dependencies |
Date: | 2017-06-23 06:37:52 |
Message-ID: | 156a4455-1688-d9d1-0277-70cddad5644f@lab.ntt.co.jp |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2017/06/23 6:28, Alvaro Herrera wrote:
> Ashutosh Bapat wrote:
>> On Mon, Jun 5, 2017 at 8:22 AM, atorikoshi
>> <torikoshi_atsushi_z2(at)lab(dot)ntt(dot)co(dot)jp> wrote:
>>> Hi,
>>>
>>> I found below formula to compute selectivities, but
>>> I think the last Probability 'P(b=?)' should be 'P(c=?)'.
>>>
>>>> P(a=?,b=?,c=?) = P(a=?,b=?) * (d + (1-d)*P(b=?))
>>>
>>>
>>> Attached patch fixes it, and it also adds some spaces
>>> following another formula which is on line 86 and
>>> computes P(a=?, b=?).
>>
>> Agree. Also using "d" for "degree of functional dependence (b=>c) as
>> well is confusing. We are already using "d" for "degree of functional
>> dependence (a=>b). Here' patch to use "d'" instead of "d".
>
> Since the surrounding text uses single quotes to talk about each letter,
> I thought it was better to use a new letter (e) so that we don't require
> the "prime" notation, which would end up being either inconsistent,
> confusing, stupid-looking, or combinations thereof.
>
> Also, your proposed text had a slight mistake: it's not (b=>c) that
> d' is for, but (a,b=>c).
>
> Pushed with those corrections.
>
> Thanks for the reports and patches!
>
Thanks!
--
Atsushi Torikoshi
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center
From | Date | Subject | |
---|---|---|---|
Next Message | Yugo Nagata | 2017-06-23 06:57:54 | Same expression more than once in partition key |
Previous Message | Ashutosh Bapat | 2017-06-23 06:37:44 | Re: Fix a typo in README.dependencies |