> On Oct 15, 2025, at 23:50, Nathan Bossart <[email protected]> wrote: > > This was first added in commit 2f17844 (v11), and AFAICT it was only used > for a variable named update_tuple_routing_needed in ExecInitModifyTable(), > which was removed by commit c5b7ba4 (v14). Any objections to removing it > now? > > -- > nathan > <v1-0001-remove-partColsUpdated.patch>
Looks like this is only one assignment to it and nobody reads it. I don’t see a reason to retain it. Maybe back patch through 14? I tried to build, and “make check” with this patch, and everything looks good. Best regards, -- Chao Li (Evan) HighGo Software Co., Ltd. https://www.highgo.com/
