Re: remove partColsUpdated

2025-10-18 Thread Michael Paquier
On Wed, Oct 15, 2025 at 08:18:41PM -0400, Tom Lane wrote: > Chao Li writes: >> On Oct 15, 2025, at 23:50, Nathan Bossart 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 wa

Re: remove partColsUpdated

2025-10-18 Thread Chao Li
> On Oct 15, 2025, at 23:50, Nathan Bossart 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? > > -

Re: remove partColsUpdated

2025-10-18 Thread Nathan Bossart
On Wed, Oct 15, 2025 at 08:18:41PM -0400, Tom Lane wrote: > Chao Li writes: >> On Oct 15, 2025, at 23:50, Nathan Bossart 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 wa

Re: remove partColsUpdated

2025-10-17 Thread Tom Lane
Chao Li writes: > On Oct 15, 2025, at 23:50, Nathan Bossart 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 i