Re: Update the comment in nodes.h to cover Cardinality

2023-12-19 Thread Richard Guo
On Tue, Dec 19, 2023 at 10:50 PM Peter Eisentraut 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? > > > > - * Typede

Re: Update the comment in nodes.h to cover Cardinality

2023-12-19 Thread Peter Eisentraut
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 a

Update the comment in nodes.h to cover Cardinality

2023-12-18 Thread Richard Guo
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