On Mon, 9 Nov 2026 at 19:30, Richard Guo <guofenglinux(at)gmail(dot)com> wrote:
>
> On Mon, Nov 9, 2026 at 2:11 PM David Rowley <dgrowleyml(at)gmail(dot)com> wrote:
> > Looking at finalize_plan(), I'd say that's correct. But shouldn't it
> > be bms_add_member() rather than bms_union()?
>
> Thanks for looking! I assume you mean bms_add_members().
Yeah. I did mean to type that.
> Attached is a patch for this fix.
>
> (Since v14 will reach EOL before this can be pushed, maybe
> back-patching to v15 is enough.)
Looks mostly ok. I'd use "add" instead of "merge" in "To fix, merge
the removed", and I expect "Backpatch-through: 14" should be 15.
David