NotificationsYou must be signed in to change notification settings
Fork5
Star27

Commit80e373c

committed

Fix statistics breakage from bgwriter/checkpointer process split.

ForwardFsyncRequest() supposed that it could only be called in regularbackends, which used to be true; but since the splitup of bgwriter andcheckpointer, it is also called in the bgwriter. We do not want to countsuch calls in pg_stat_bgwriter.buffers_backend statistics, so fix thingsso that they aren't.(It's worth noting here that this implies an alarmingly large increase inthe expected amount of cross-process fsync request traffic, which may wellmean that the process splitup was not such a hot idea.)

1 parent4a9c30a commit80e373cCopy full SHA for 80e373c

File tree

1 file changed

-2

lines changed

src/backend/postmaster
- checkpointer.c

1 file changed

-2

lines changed

`‎src/backend/postmaster/checkpointer.c‎`

Lines changed: 4 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -1132,7 +1132,8 @@ ForwardFsyncRequest(RelFileNode rnode, ForkNumber forknum, BlockNumber segno)`
`1132`	`1132`	`LWLockAcquire(CheckpointerCommLock,LW_EXCLUSIVE);`
`1133`	`1133`
`1134`	`1134`	`/* Count all backend writes regardless of if they fit in the queue */`
`1135`		`-CheckpointerShmem->num_backend_writes++;`
	`1135`	`+if (!AmBackgroundWriterProcess())`
	`1136`	`+CheckpointerShmem->num_backend_writes++;`
`1136`	`1137`
`1137`	`1138`	`/*`
`1138`	`1139`	`* If the checkpointer isn't running or the request queue is full, the`
`@@ -1147,7 +1148,8 @@ ForwardFsyncRequest(RelFileNode rnode, ForkNumber forknum, BlockNumber segno)`
`1147`	`1148`	`* Count the subset of writes where backends have to do their own`
`1148`	`1149`	`* fsync`
`1149`	`1150`	`*/`
`1150`		`-CheckpointerShmem->num_backend_fsync++;`
	`1151`	`+if (!AmBackgroundWriterProcess())`
	`1152`	`+CheckpointerShmem->num_backend_fsync++;`
`1151`	`1153`	`LWLockRelease(CheckpointerCommLock);`
`1152`	`1154`	`return false;`
`1153`	`1155`	`}`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit80e373c

File tree

1 file changed

1 file changed

`‎src/backend/postmaster/checkpointer.c‎`

0 commit comments