Re: BitmapHeapScan streaming read user and prelim refactoring - Mailing list pgsql-hackers

From Thomas Munro
Subject Re: BitmapHeapScan streaming read user and prelim refactoring
Date
Msg-id CA+hUKGLmu0w8K=Q2OvYAoiaHDL_UfrJFBz6ktCrvfy297SL2Bg@mail.gmail.com
Whole thread Raw
In response to Re: BitmapHeapScan streaming read user and prelim refactoring  (Melanie Plageman <melanieplageman@gmail.com>)
Responses Re: BitmapHeapScan streaming read user and prelim refactoring
List pgsql-hackers
On Sat, Sep 28, 2024 at 8:13 AM Melanie Plageman
<melanieplageman@gmail.com> wrote:
> For the top-level TableScanDescData, I suggest we use a union with the
> members of each scan type in it in anonymous structs (see 0001).

Just by the way, you can't use anonymous structs or unions in C99
(that was added to C11), but most compilers accept them silently
unless you use eg -std=c99.  Some buildfarm animal or other would
bleat about that (ask me how I know), but out CI doesn't pick it up
:-(  Maybe we should fix that.  That's why you see a lot of code
around that accesses unions through a member with a name like "u",
because the author was shaking her fist at the god of standards and
steadfastly refusing to think of a name.  It's a shame in cases where
you want to retrofit a union into existing code without having to
adjust the code that accesses one of the members...



pgsql-hackers by date:

Previous
From: Daniel Gustafsson
Date:
Subject: Re: Changing the state of data checksums in a running cluster
Next
From: Daniel Gustafsson
Date:
Subject: Re: pgindent fails with perl 5.40