diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2023-03-29 09:13:57 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2023-03-29 09:13:57 -0400 |
| commit | 58c9600a9f5818f63712ae4262143c42517dcba7 (patch) | |
| tree | 37e998d858acca5dc39e08d39f51ee27c6aa9561 /src/include/storage/bufmgr.h | |
| parent | 7081ac46ace8c459966174400b53418683c9fe5c (diff) | |
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
Diffstat (limited to 'src/include/storage/bufmgr.h')
| -rw-r--r-- | src/include/storage/bufmgr.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/include/storage/bufmgr.h b/src/include/storage/bufmgr.h index b8a18b8081f..73762cb1ec8 100644 --- a/src/include/storage/bufmgr.h +++ b/src/include/storage/bufmgr.h @@ -182,7 +182,6 @@ extern bool HoldingBufferPinThatDelaysRecovery(void); extern void AbortBufferIO(void); -extern void BufmgrCommit(void); extern bool BgBufferSync(struct WritebackContext *wb_context); extern void TestForOldSnapshot_impl(Snapshot snapshot, Relation relation); |