On 2026-Nov-02, Tender Wang wrote:

> As you noted, adding a field to JoinPathExtraData raises an ABI
> concern, so that version of the fix might need to be master-only.

But nothing allocates JoinPathExtraData anywhere except
add_paths_to_joinrel in its stack; everybody else just receives a
pointer to that.  Even the hooks for it receive it as a pointer.  I
think the ABI concern is overblown in this case; just make sure to add
the new field at the end.

-- 
Álvaro Herrera        Breisgau, Deutschland  —  https://www.EnterpriseDB.com/
"Find a bug in a program, and fix it, and the program will work today.
Show the program how to find and fix a bug, and the program
will work forever" (Oliver Silfridge)


Reply via email to