Re: Update the comment in nodes.h to cover Cardinality - Mailing list pgsql-hackers

From Richard Guo
Subject Re: Update the comment in nodes.h to cover Cardinality
Date
Msg-id CAMbWs49L3Rj=uN0U4iXXL8qYf9-dAsKcjQF_MwNNdtx2rau37Q@mail.gmail.com
Whole thread Raw
In response to Re: Update the comment in nodes.h to cover Cardinality  (Peter Eisentraut <peter@eisentraut.org>)
List pgsql-hackers

On Tue, Dec 19, 2023 at 10:50 PM Peter Eisentraut <peter@eisentraut.org> wrote:
On 19.12.23 07:23, Richard Guo wrote:
> By chance I discovered that the comment for the typedefs of "double"s
> does not cover Cardinality.  Should we update that comment accordingly,
> maybe something like below?
>
> - * Typedefs for identifying qualifier selectivities and plan costs as such.
> - * These are just plain "double"s, but declaring a variable as Selectivity
> - * or Cost makes the intent more obvious.
> + * Typedefs for identifying qualifier selectivities, plan costs and
> + * estimated rows or other count as such.  These are just plain "double"s,
> + * but declaring a variable as Selectivity, Cost or Cardinality makes the
> + * intent more obvious.

Fixed, thanks.

Thanks for the fix!

Thanks
Richard

pgsql-hackers by date:

Previous
From: Tomas Vondra
Date:
Subject: Re: Use of additional index columns in rows filtering
Next
From: Tomas Vondra
Date:
Subject: Re: Use of additional index columns in rows filtering