Thread: Typos in the comment for the estimate_multivariate_ndistinct()
Hi,
While reading the estimate_multivariate_ndistinct(),
I think "If a match it found, *varinfos is
* updated to remove the list of matched varinfos"should be "If a match is found, *varinfos is
* updated to remove the list of matched varinfos"
* updated to remove the list of matched varinfos"
I've attached a patch for that.
--
Thanks,
Tender Wang
Attachment
Tender Wang <tndrwang@gmail.com> 于2025年4月14日周一 21:34写道：
Hi,While reading the estimate_multivariate_ndistinct(),I think "If a match it found, *varinfos is* updated to remove the list of matched varinfos"should be "If a match is found, *varinfos is
* updated to remove the list of matched varinfos"I've attached a patch for that.
Hi Alvaro,
Can you help me double-check this?
Thanks,
Tender Wang
> On 14 Apr 2025, at 15:34, Tender Wang <tndrwang@gmail.com> wrote: > > Hi, > > While reading the estimate_multivariate_ndistinct(), > I think "If a match it found, *varinfos is > * updated to remove the list of matched varinfos" > should be "If a match is found, *varinfos is > * updated to remove the list of matched varinfos" > I've attached a patch for that. Seems like a correct change. -- Daniel Gustafsson
Daniel Gustafsson <daniel@yesql.se> 于2025年4月16日周三 22:20写道：
> On 14 Apr 2025, at 15:34, Tender Wang <tndrwang@gmail.com> wrote:
>
> Hi,
>
> While reading the estimate_multivariate_ndistinct(),
> I think "If a match it found, *varinfos is
> * updated to remove the list of matched varinfos"
> should be "If a match is found, *varinfos is
> * updated to remove the list of matched varinfos"
> I've attached a patch for that.
Seems like a correct change.
Thanks for checking.
Thanks,
Tender Wang