NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit9ae6c11

committed

Code review for multixact bugfix

Reword messages, rename a confusingly named function.Per Robert Haas.

1 parent942542c commit9ae6c11Copy full SHA for 9ae6c11

File tree

1 file changed

-13

lines changed

src/backend/access/transam
- multixact.c

1 file changed

-13

lines changed

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

Lines changed: 9 additions & 13 deletions

Original file line number	Diff line number	Diff line change
`@@ -345,7 +345,7 @@ static void ExtendMultiXactMember(MultiXactOffset offset, int nmembers);`
`345`	`345`	`staticvoidDetermineSafeOldestOffset(MultiXactIdoldestMXact);`
`346`	`346`	`staticboolMultiXactOffsetWouldWrap(MultiXactOffsetboundary,`
`347`	`347`	`MultiXactOffsetstart,uint32distance);`
`348`		`-staticMultiXactOffsetread_offset_for_multi(MultiXactIdmulti);`
	`348`	`+staticMultiXactOffsetfind_multixact_start(MultiXactIdmulti);`
`349`	`349`	`staticvoidWriteMZeroPageXlogRec(intpageno,uint8info);`
`350`	`350`
`351`	`351`
`@@ -1079,12 +1079,12 @@ GetNewMultiXactId(int nmembers, MultiXactOffset *offset)`
`1079`	`1079`	`ereport(ERROR,`
`1080`	`1080`	`(errcode(ERRCODE_PROGRAM_LIMIT_EXCEEDED),`
`1081`	`1081`	`errmsg("multixact \"members\" limit exceeded"),`
`1082`		`-errdetail_plural("This command would create a multixact with %u members,which exceeds remaining space(%u member.)",`
`1083`		`-"This command would create a multixact with %u members,which exceeds remaining space(%u members.)",`
	`1082`	`+errdetail_plural("This command would create a multixact with %u members,but the remaining spaceis only enough for%u member.",`
	`1083`	`+"This command would create a multixact with %u members,but the remaining spaceis only enough for%u members.",`
`1084`	`1084`	`MultiXactState->offsetStopLimit-nextOffset-1,`
`1085`	`1085`	`nmembers,`
`1086`	`1086`	`MultiXactState->offsetStopLimit-nextOffset-1),`
`1087`		`-errhint("Execute a database-wide VACUUM in database with OID %u, with reduced vacuum_multixact_freeze_min_age and vacuum_multixact_freeze_table_age settings.",`
	`1087`	`+errhint("Execute a database-wide VACUUM in database with OID %u with reduced vacuum_multixact_freeze_min_age and vacuum_multixact_freeze_table_age settings.",`
`1088`	`1088`	`MultiXactState->oldestMultiXactDB)));`
`1089`	`1089`	`elseif (MultiXactOffsetWouldWrap(MultiXactState->offsetStopLimit,`
`1090`	`1090`	`nextOffset,`
`@@ -1094,7 +1094,7 @@ GetNewMultiXactId(int nmembers, MultiXactOffset *offset)`
`1094`	`1094`	`errmsg("database with OID %u must be vacuumed before %d more multixact members are used",`
`1095`	`1095`	`MultiXactState->oldestMultiXactDB,`
`1096`	`1096`	`MultiXactState->offsetStopLimit-nextOffset+nmembers),`
`1097`		`-errhint("Execute a database-wide VACUUM in that database, with reduced vacuum_multixact_freeze_min_age and vacuum_multixact_freeze_table_age settings.")));`
	`1097`	`+errhint("Execute a database-wide VACUUM in that database with reduced vacuum_multixact_freeze_min_age and vacuum_multixact_freeze_table_age settings.")));`
`1098`	`1098`
`1099`	`1099`	`ExtendMultiXactMember(nextOffset,nmembers);`
`1100`	`1100`
`@@ -2506,7 +2506,7 @@ DetermineSafeOldestOffset(MultiXactId oldestMXact)`
`2506`	`2506`	`* one-segment hole at a minimum. We start spewing warnings a few`
`2507`	`2507`	`* complete segments before that.`
`2508`	`2508`	`*/`
`2509`		`-oldestOffset=read_offset_for_multi(oldestMXact);`
	`2509`	`+oldestOffset=find_multixact_start(oldestMXact);`
`2510`	`2510`	`/* move back to start of the corresponding segment */`
`2511`	`2511`	`oldestOffset-=oldestOffset /MULTIXACT_MEMBERS_PER_PAGE*SLRU_PAGES_PER_SEGMENT;`
`2512`	`2512`
`@@ -2562,20 +2562,16 @@ MultiXactOffsetWouldWrap(MultiXactOffset boundary, MultiXactOffset start,`
`2562`	`2562`	`*-----------------------------------------------------------------------`
`2563`	`2563`	`*/`
`2564`	`2564`	`if (start<boundary)`
`2565`		`-{`
`2566`	`2565`	`returnfinish >=boundary\|\|finish<start;`
`2567`		`-}`
`2568`	`2566`	`else`
`2569`		`-{`
`2570`	`2567`	`returnfinish >=boundary&&finish<start;`
`2571`		`-}`
`2572`	`2568`	`}`
`2573`	`2569`
`2574`	`2570`	`/*`
`2575`		`- *Read the offset of thefirst member of thegivenmultixact.`
	`2571`	`+ *Find thestartingoffset of the givenMultiXactId.`
`2576`	`2572`	`*/`
`2577`	`2573`	`staticMultiXactOffset`
`2578`		`-read_offset_for_multi(MultiXactIdmulti)`
	`2574`	`+find_multixact_start(MultiXactIdmulti)`
`2579`	`2575`	`{`
`2580`	`2576`	`MultiXactOffsetoffset;`
`2581`	`2577`	`intpageno;`
`@@ -2728,7 +2724,7 @@ TruncateMultiXact(void)`
`2728`	`2724`	`* First, compute the safe truncation point for MultiXactMember. This is`
`2729`	`2725`	`* the starting offset of the oldest multixact.`
`2730`	`2726`	`*/`
`2731`		`-oldestOffset=read_offset_for_multi(oldestMXact);`
	`2727`	`+oldestOffset=find_multixact_start(oldestMXact);`
`2732`	`2728`
`2733`	`2729`	`/*`
`2734`	`2730`	`* To truncate MultiXactMembers, we need to figure out the active page`

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!

Commit9ae6c11

File tree

1 file changed

1 file changed

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

0 commit comments