From 978b032d1f39fcfe9b7e895ab313913d97db2268 Mon Sep 17 00:00:00 2001 From: Fujii Masao Date: Thu, 25 Apr 2019 23:43:48 +0900 Subject: [PATCH] Fix function names in comments. Commit 3eb77eba5a renamed some functions, but forgot to update some comments referencing to those functions. This commit fixes those function names in the comments. Kyotaro Horiguchi --- src/backend/postmaster/checkpointer.c | 2 +- src/backend/storage/smgr/md.c | 2 +- src/backend/storage/sync/sync.c | 8 ++++---- src/common/rmtree.c | 2 +- 4 files changed, 7 insertions(+), 7 deletions(-) diff --git a/src/backend/postmaster/checkpointer.c b/src/backend/postmaster/checkpointer.c index a1e04239ad0..13f152b4731 100644 --- a/src/backend/postmaster/checkpointer.c +++ b/src/backend/postmaster/checkpointer.c @@ -137,7 +137,7 @@ typedef struct static CheckpointerShmemStruct *CheckpointerShmem; -/* interval for calling AbsorbFsyncRequests in CheckpointWriteDelay */ +/* interval for calling AbsorbSyncRequests in CheckpointWriteDelay */ #define WRITES_PER_ABSORB 1000 /* diff --git a/src/backend/storage/smgr/md.c b/src/backend/storage/smgr/md.c index 61a8f11469b..52ca6eeb28f 100644 --- a/src/backend/storage/smgr/md.c +++ b/src/backend/storage/smgr/md.c @@ -952,7 +952,7 @@ register_forget_request(RelFileNodeBackend rnode, ForkNumber forknum, } /* - * ForgetDatabaseFsyncRequests -- forget any fsyncs and unlinks for a DB + * ForgetDatabaseSyncRequests -- forget any fsyncs and unlinks for a DB */ void ForgetDatabaseSyncRequests(Oid dbid) diff --git a/src/backend/storage/sync/sync.c b/src/backend/storage/sync/sync.c index f77519d7d15..096735c8071 100644 --- a/src/backend/storage/sync/sync.c +++ b/src/backend/storage/sync/sync.c @@ -75,7 +75,7 @@ static MemoryContext pendingOpsCxt; /* context for the above */ static CycleCtr sync_cycle_ctr = 0; static CycleCtr checkpoint_cycle_ctr = 0; -/* Intervals for calling AbsorbFsyncRequests */ +/* Intervals for calling AbsorbSyncRequests */ #define FSYNCS_PER_ABSORB 10 #define UNLINKS_PER_ABSORB 10 @@ -215,9 +215,9 @@ SyncPostCheckpoint(void) pfree(entry); /* - * As in ProcessFsyncRequests, we don't want to stop absorbing fsync + * As in ProcessSyncRequests, we don't want to stop absorbing fsync * requests for along time when there are many deletions to be done. - * We can safely call AbsorbFsyncRequests() at this point in the loop + * We can safely call AbsorbSyncRequests() at this point in the loop * (note it might try to delete list entries). */ if (--absorb_counter <= 0) @@ -284,7 +284,7 @@ ProcessSyncRequests(void) * eventually wrap the counter around to the point where an old entry * might appear new, causing us to skip it, possibly allowing a checkpoint * to succeed that should not have. To forestall wraparound, any time the - * previous ProcessFsyncRequests() failed to complete, run through the + * previous ProcessSyncRequests() failed to complete, run through the * table and forcibly set cycle_ctr = sync_cycle_ctr. * * Think not to merge this loop with the main loop, as the problem is diff --git a/src/common/rmtree.c b/src/common/rmtree.c index 3052d013ee9..b31da3adff8 100644 --- a/src/common/rmtree.c +++ b/src/common/rmtree.c @@ -68,7 +68,7 @@ rmtree(const char *path, bool rmtopdir) * This is not an academic possibility. One scenario where this * happens is when bgwriter has a pending unlink request for a file in * a database that's being dropped. In dropdb(), we call - * ForgetDatabaseFsyncRequests() to flush out any such pending unlink + * ForgetDatabaseSyncRequests() to flush out any such pending unlink * requests, but because that's asynchronous, it's not guaranteed that * the bgwriter receives the message in time. */