Re: remove partColsUpdated

On Wed, Oct 15, 2025 at 08:18:41PM -0400, Tom Lane wrote:
> Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com> writes:
>> On Oct 15, 2025, at 23:50, Nathan Bossart <nathandbossart(at)gmail(dot)com> 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?
>
>> Looks like this is only one assignment to it and nobody reads it. I
>> don’t see a reason to retain it.
>
> I just had a look through https://codesearch.debian.net/ and couldn't
> find any evidence that any extensions are using it, so +1 to remove.

Thanks, will go commit it now.

>> Maybe back patch through 14?
>
> Certainly not. That would cause an ABI break for any extension that
> touches later fields in PlannerInfo or ModifyTable. We don't expect
> extensions to get recompiled for minor releases.

Even if it wasn't an ABI break, there's very little upside to back-patching
small stuff like this that's not hurting anything. The risk/reward ratio
is not favorable.

--
nathan