Commitd265cd2

committed

Use SetInstallXLogFileSegmentActive() in more places in xlog.c

This reduces the code paths where XLogCtl->InstallXLogFileSegmentActiveis directly touched, and this wrapper function does the same thing asthe original code replaced by the function call.Author: Bharath RupireddyReviewed-by: Nathan BossartDiscussion:https://postgr.es/m/CALj2ACVhkf-bC5CX-=6iBUfkO5GqmBntQH+m=HpY0iQ=-g1pRg@mail.gmail.com

1 parent93f2349 commitd265cd2Copy full SHA for d265cd2

File tree

-6

lines changed

-6

lines changed

Lines changed: 2 additions & 6 deletions

Original file line number	Diff line number	Diff line change
`@@ -4501,9 +4501,7 @@ BootStrapXLOG(void)`
`4501`	`4501`	`pg_crc32ccrc;`
`4502`	`4502`
`4503`	`4503`	`/* allow ordinary WAL segment creation, like StartupXLOG() would */`
`4504`		`-LWLockAcquire(ControlFileLock,LW_EXCLUSIVE);`
`4505`		`-XLogCtl->InstallXLogFileSegmentActive= true;`
`4506`		`-LWLockRelease(ControlFileLock);`
	`4504`	`+SetInstallXLogFileSegmentActive();`
`4507`	`4505`
`4508`	`4506`	`/*`
`4509`	`4507`	`* Select a hopefully-unique system identifier code for this installation.`
`@@ -5368,9 +5366,7 @@ StartupXLOG(void)`
`5368`	`5366`	`* Allow ordinary WAL segment creation before possibly switching to a new`
`5369`	`5367`	`* timeline, which creates a new segment, and after the last ReadRecord().`
`5370`	`5368`	`*/`
`5371`		`-LWLockAcquire(ControlFileLock,LW_EXCLUSIVE);`
`5372`		`-XLogCtl->InstallXLogFileSegmentActive= true;`
`5373`		`-LWLockRelease(ControlFileLock);`
	`5369`	`+SetInstallXLogFileSegmentActive();`
`5374`	`5370`
`5375`	`5371`	`/*`
`5376`	`5372`	`* Consider whether we need to assign a new timeline ID.`

Comments

(0)