| From: | Peter Eisentraut <peter(at)eisentraut(dot)org> |
|---|---|
| To: | Richard Guo <guofenglinux(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Update the comment in nodes.h to cover Cardinality |
| Date: | 2023-12-19 14:50:37 |
| Message-ID: | 7b975ded-915a-4e9d-aaf9-f0e75ce12e0d@eisentraut.org |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
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.
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Junwang Zhao | 2023-12-19 14:51:03 | Re: Transaction timeout |
| Previous Message | Jelte Fennema-Nio | 2023-12-19 14:44:43 | Re: Add new for_each macros for iterating over a List that do not require ListCell pointer |