Make Bitmapsets be valid Nodes.
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 13 Nov 2022 15:22:45 +0000 (10:22 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 13 Nov 2022 15:22:45 +0000 (10:22 -0500)
commit5e1f3b9ebf6e51f2c60abe9aba969c28a3b0f6c6
treed12f6c444f7ca89fe26e69695884412d79703b01
parent9c7eb9d85a5687d4f9624c40ee3a508d8c7de35a
Make Bitmapsets be valid Nodes.

Add a NodeTag field to struct Bitmapset.  This is free because of
alignment considerations on 64-bit hardware.  While it adds some
space on 32-bit machines, we aren't optimizing for that case anymore.
The advantage is that data structures such as Lists of Bitmapsets
are now first-class objects to the Node infrastructure, and don't
require special-case code to handle.

This patch includes removal of one such special case, in indxpath.c:
bms_equal_any() can now be replaced by list_member().  There may be
more existing code that could be simplified, but I didn't look very
hard.  We also get to drop the read_write_ignore annotations on a
couple of RelOptInfo fields.

The outfuncs/readfuncs support is arranged so that nothing changes
in the string representation of a Bitmapset field; therefore, this
doesn't need a catversion bump.

Amit Langote and Tom Lane

Discussion: https://postgr.es/m/109089.1668197158@sss.pgh.pa.us
12 files changed:
src/backend/nodes/Makefile
src/backend/nodes/bitmapset.c
src/backend/nodes/copyfuncs.c
src/backend/nodes/equalfuncs.c
src/backend/nodes/gen_node_support.pl
src/backend/nodes/outfuncs.c
src/backend/nodes/read.c
src/backend/nodes/readfuncs.c
src/backend/optimizer/path/indxpath.c
src/include/nodes/bitmapset.h
src/include/nodes/meson.build
src/include/nodes/pathnodes.h