Commit00ee6c7

committed

autovacuum: Fix polarity of "wraparound" variable

Commit0d83138 inadvertently reversed the meaning of thewraparound variable. This causes vacuums which are not required forwraparound to wait for locks to be acquired, and what is worse, itallows wraparound vacuums to skip locked pages.Bug reported by Jeff Janes inhttp://www.postgresql.org/message-id/CAMkU=1xmTEiaY=5oMHsSQo5vd9V1Ze4kNLL0qN2eH0P_GXOaYw@mail.gmail.comAnalysis and patch by Kyotaro HORIGUCHI

1 parentc02ef23 commit00ee6c7Copy full SHA for 00ee6c7

File tree

-1

lines changed

-1

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -2526,7 +2526,7 @@ table_recheck_autovac(Oid relid, HTAB *table_toast_map,`
`2526`	`2526`	`tab->at_vacoptions=VACOPT_SKIPTOAST \|`
`2527`	`2527`	`(dovacuum ?VACOPT_VACUUM :0) \|`
`2528`	`2528`	`(doanalyze ?VACOPT_ANALYZE :0) \|`
`2529`		`-(wraparound ?VACOPT_NOWAIT :0);`
	`2529`	`+(!wraparound ?VACOPT_NOWAIT :0);`
`2530`	`2530`	`tab->at_params.freeze_min_age=freeze_min_age;`
`2531`	`2531`	`tab->at_params.freeze_table_age=freeze_table_age;`
`2532`	`2532`	`tab->at_params.multixact_freeze_min_age=multixact_freeze_min_age;`

Comments

(0)