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

remove partColsUpdated

2025-10-17 Thread Nathan Bossart
17 00:00:00 2001 From: Nathan Bossart Date: Wed, 15 Oct 2025 10:40:18 -0500 Subject: [PATCH v1 1/1] remove partColsUpdated --- src/backend/optimizer/plan/createplan.c | 4 src/backend/optimizer/plan/planner.c| 2 -- src/backend/optimizer/util/inherit.c| 10 -- src/back

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