Remove empty function BufmgrCommit().
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 29 Mar 2023 13:13:57 +0000 (09:13 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 29 Mar 2023 13:13:57 +0000 (09:13 -0400)
commit58c9600a9f5818f63712ae4262143c42517dcba7
tree37e998d858acca5dc39e08d39f51ee27c6aa9561
parent7081ac46ace8c459966174400b53418683c9fe5c
Remove empty function BufmgrCommit().

This function has been a no-op for over a decade.  Even if bufmgr
regains a need to be called during commit, it seems unlikely that
the most appropriate call points would be precisely here, so it's not
doing us much good as a placeholder either.  Now, removing it probably
doesn't save any noticeable number of cycles --- but the main call is
inside the commit critical section, and the less work done there the
better.

Matthias van de Meent

Discussion: https://postgr.es/m/CAEze2Wi1=tLKbxZnXzcD+8fYKyKqBtivVakLQC_mYBsP4Y8qVA@mail.gmail.com
src/backend/access/transam/xact.c
src/backend/storage/buffer/bufmgr.c
src/include/storage/bufmgr.h