On 11.11.22 21:05, Tom Lane wrote:
> Per the discussion at [1], it seems like it'd be a good idea to make
> Bitmapsets into full-fledged, tagged Nodes, so that we could do things
> like print or copy lists of them without special-case logic. The
> extra space for the NodeTag is basically free due to alignment
> considerations, at least on 64-bit hardware.
>
> Attached is a cleaned-up version of Amit's patch v24-0003 at [2].
> I fixed the problems with not always tagging Bitmapsets, and changed
> the outfuncs/readfuncs logic so that Bitmapsets still print exactly
> as they did before (thus, this doesn't require a catversion bump).
This looks good to me.
