On Mon, 9 Nov 2026 at 19:30, Richard Guo <[email protected]> wrote:
>
> On Mon, Nov 9, 2026 at 2:11 PM David Rowley <[email protected]> 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


Reply via email to