Thread: BufmgrCommit no-op since 2008, remaining uses?

Hi,

Whilst reading up on the transaction commit code, I noticed the following lines:

  /* Tell bufmgr and smgr to prepare for commit */
  BufmgrCommit();

BufmgrCommit does exactly nothing; it is an empty function and has
been since commit 33960006 in late 2008 when it stopped calling
smgrcommit().

All two usages of the function (in our code base) seem to be in
xact.c. Are we maintaining it for potential future use, or can the
function be removed?

Kind regards,

Matthias van de Meent

Matthias van de Meent <boekewurm+postgres@gmail.com> writes:
> BufmgrCommit does exactly nothing; it is an empty function and has
> been since commit 33960006 in late 2008 when it stopped calling
> smgrcommit().
> All two usages of the function (in our code base) seem to be in
> xact.c. Are we maintaining it for potential future use, or can the
> function be removed?

Seems reasonable.  Even if bufmgr grew a new need to be called
during commit, it would quite possibly need to be called from
a different spot; so I doubt that the function is useful even
as a placeholder.

            regards, tom lane

On Wed, 29 Mar 2023 at 14:12, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>
> Matthias van de Meent <boekewurm+postgres@gmail.com> writes:
> > BufmgrCommit does exactly nothing; it is an empty function and has
> > been since commit 33960006 in late 2008 when it stopped calling
> > smgrcommit().
> > All two usages of the function (in our code base) seem to be in
> > xact.c. Are we maintaining it for potential future use, or can the
> > function be removed?
>
> Seems reasonable.  Even if bufmgr grew a new need to be called
> during commit, it would quite possibly need to be called from
> a different spot; so I doubt that the function is useful even
> as a placeholder.

Then, the attached trivial patch removes the function and all
references I could find.

Kind regards,

Matthias van de Meent

Matthias van de Meent <boekewurm+postgres@gmail.com> writes:
> Then, the attached trivial patch removes the function and all
> references I could find.

Pushed after a bit of fooling with adjacent comments.

            regards, tom lane