Update the comment in nodes.h to cover Cardinality
Richard Guo <guofenglinux@gmail.com>
From: Richard Guo <guofenglinux@gmail.com>
To: PostgreSQL-development <pgsql-hackers@postgresql.org>
Cc: Peter Eisentraut <peter@eisentraut.org>
Date: 2023-12-19T06:23:25Z
Lists: pgsql-hackers
Attachments
- v1-0001-Add-comment-for-Cardinality-typedef.patch (application/octet-stream) patch v1-0001
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. Thanks Richard
Commits
-
Update comment for Cardinality typedef
- 2a607fb822a2 17.0 landed