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"
I've attached a patch for that.
--
Thanks,
Tender Wang
Attachments:
0001-Fix-a-typo.patchtext/plain; charset=US-ASCII; name=0001-Fix-a-typo.patchDownload+1-2
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