NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit2b88fdd

committed

Track SLRU page hits in SimpleLruReadPage_ReadOnly

SLRU page hits were tracked only in SimpleLruReadPage, but that's notenough because we may hit the page in SimpleLruReadPage_ReadOnly inwhich case we don't call SimpleLruReadPage at all.Reported-by: Kuntal GhoshDiscussion:https://postgr.es/m/20200119143707.gyinppnigokesjok@development

1 parent91c4054 commit2b88fddCopy full SHA for 2b88fdd

File tree

1 file changed

-0

lines changed

src/backend/access/transam
- slru.c

1 file changed

-0

lines changed

`‎src/backend/access/transam/slru.c`

Lines changed: 4 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -491,6 +491,10 @@ SimpleLruReadPage_ReadOnly(SlruCtl ctl, int pageno, TransactionId xid)`
`491`	`491`	`{`
`492`	`492`	`/* See comments for SlruRecentlyUsed macro */`
`493`	`493`	`SlruRecentlyUsed(shared,slotno);`
	`494`	`+`
	`495`	`+/* update the stats counter of pages found in the SLRU */`
	`496`	`+pgstat_count_slru_page_hit(ctl);`
	`497`	`+`
`494`	`498`	`returnslotno;`
`495`	`499`	`}`
`496`	`500`	`}`

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!

Commit2b88fdd

File tree

1 file changed

1 file changed

`‎src/backend/access/transam/slru.c`

0 commit comments