On Mon, Apr 6, 2026 at 12:50 PM Melanie Plageman
<melanieplageman@gmail.com> wrote:
>
> I cleaned up the first patch in the set that refactors index-only and
> index scan to use this pattern and realized that I wasn't sure what to
> do about the duplication between index-only and index scans for these
> functions.
I changed my mind about the duplication: I think it's fine because
there just aren't that many lines of code. Attached v10 has a
preparatory commit that adds an assert-only check to shm_toc_insert()
to make sure the key hasn't already been inserted. I thought it would
be helpful since many of the magic numbers for parallel executor
communication are spread around.
- Melanie
