shihao zhong <[email protected]> writes:
> On Tue, Jul 28, 2026 at 11:44 PM shveta malik <[email protected]> wrote:
>> On Tue, Jul 28, 2026 at 10:30 AM Peter Smith <[email protected]> wrote:
>>> FWIW, I detected a missing `list_free`, which seems to have been
>>> accidentally left out from commit 923def9.
>> Okay since list_free for both relids and relids_with_rf are there, I
>> think list_free for 'relids_with_rf' also make sense. So LGTM.
AFAICS, that function is executed at most once per DDL command,
in a command-lifetime memory context. I think the right fix
is not to add another list_free but to remove the misleadingly
useless ones that are there.
regards, tom lane
