NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit32d6287

committed

Get heap page max offset with buffer lock held.

On further reflection it seems better to call PageGetMaxOffsetNumber()after acquiring a buffer lock on the page. This shouldn't reallymatter, but doing it this way is cleaner.Follow-up to commit4228817.Backpatch: 12-, just like commit4228817

1 parent4228817 commit32d6287Copy full SHA for 32d6287

File tree

1 file changed

-2

lines changed

src/backend/access/heap
- heapam.c

1 file changed

-2

lines changed

`‎src/backend/access/heap/heapam.c`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -7075,10 +7075,10 @@ heap_compute_xid_horizon_for_tuples(Relation rel,`
`7075`	`7075`	`xid_horizon_prefetch_buffer(rel,&prefetch_state,1);`
`7076`	`7076`	`#endif`
`7077`	`7077`
	`7078`	`+LockBuffer(buf,BUFFER_LOCK_SHARE);`
	`7079`	`+`
`7078`	`7080`	`page=BufferGetPage(buf);`
`7079`	`7081`	`maxoff=PageGetMaxOffsetNumber(page);`
`7080`		`-`
`7081`		`-LockBuffer(buf,BUFFER_LOCK_SHARE);`
`7082`	`7082`	`}`
`7083`	`7083`
`7084`	`7084`	`/*`

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!

Commit32d6287

File tree

1 file changed

1 file changed

`‎src/backend/access/heap/heapam.c`

0 commit comments