Re: remove partColsUpdated - Mailing list pgsql-hackers

From Tom Lane
Subject Re: remove partColsUpdated
Date
Msg-id 1339305.1760573921@sss.pgh.pa.us
Whole thread Raw
In response to Re: remove partColsUpdated  (Chao Li <li.evan.chao@gmail.com>)
Responses Re: remove partColsUpdated
List pgsql-hackers
Chao Li <li.evan.chao@gmail.com> writes:
> On Oct 15, 2025, at 23:50, Nathan Bossart <nathandbossart@gmail.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.

> 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.

            regards, tom lane



pgsql-hackers by date:

Previous
From: Peter Smith
Date:
Subject: Re: POC: enable logical decoding when wal_level = 'replica' without a server restart
Next
From: Michael Paquier
Date:
Subject: Re: [BUG] temporary file usage report with extended protocol and unnamed portals