Re: Remove unused isCommit parameter from AtEOXact_LocalBuffers

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: zengman <zengman(at)halodbtech(dot)com>
Cc: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Remove unused isCommit parameter from AtEOXact_LocalBuffers
Date: 2026-02-03 14:45:49
Message-ID: 1226920.1770129949@sss.pgh.pa.us
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

"=?ISO-8859-1?B?emVuZ21hbg==?=" <zengman(at)halodbtech(dot)com> writes:
> Commit fdd13f156814f81732c188788ab1b7b14c59f4da removed the above code, but the `isCommit` parameter was left intact.
> I intend to remove this unused parameter, and also clean up the call site in `AtEOXact_Buffers` since it invokes `AtEOXact_LocalBuffers`.

I think we should reject this as useless code churn. The parameter
was needed in the past and might be needed again in the future.
It's fairly common for other AtEOXact functions to take an isCommit
flag, so I don't find it surprising for these to have one.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2026-02-03 14:56:03 Re: Get rid of the pre-C11 _Alignof define in zic.c
Previous Message Tom Lane 2026-02-03 14:41:54 Re: Add LIMIT option to COPY FROM