NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit7be47c5

committed

Fix incorrect math in DetermineSafeOldestOffset.

The old formula didn't have enough parentheses, so it would do the wrongthing, and it used / rather than % to find a remainder. The effect ofthese oversights is that the stop point chosen by the logic introduced incommitb69bf30 might be rathermeaningless.Thomas Munro, reviewed by Kevin Grittner, with a whitespace tweak by me.

1 parent82ec7c9 commit7be47c5Copy full SHA for 7be47c5

File tree

1 file changed

-1

lines changed

src/backend/access/transam
- multixact.c

1 file changed

-1

lines changed

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

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -2495,7 +2495,8 @@ DetermineSafeOldestOffset(MultiXactId oldestMXact)`
`2495`	`2495`	`*/`
`2496`	`2496`	`oldestOffset=find_multixact_start(oldestMXact);`
`2497`	`2497`	`/* move back to start of the corresponding segment */`
`2498`		`-oldestOffset-=oldestOffset /MULTIXACT_MEMBERS_PER_PAGE*SLRU_PAGES_PER_SEGMENT;`
	`2498`	`+oldestOffset-=oldestOffset %`
	`2499`	`+(MULTIXACT_MEMBERS_PER_PAGE*SLRU_PAGES_PER_SEGMENT);`
`2499`	`2500`
`2500`	`2501`	`LWLockAcquire(MultiXactGenLock,LW_EXCLUSIVE);`
`2501`	`2502`	`/* always leave one segment before the wraparound point */`

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!

Commit7be47c5

File tree

1 file changed

1 file changed

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

0 commit comments