Commite9e441c

committed

Prevent mark as deleted and as 'has free space' page in bloom module

Vacuum might put page into list of pages with some free space and mark asdeleted at the same time.

1 parent80afb62 commite9e441cCopy full SHA for e9e441c

File tree

-2

lines changed

-2

lines changed

Lines changed: 7 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -108,15 +108,20 @@ blbulkdelete(IndexVacuumInfo info, IndexBulkDeleteResult stats,`
`108`	`108`	`Assert(itupPtr==BloomPageGetTuple(&state,page,`
`109`	`109`	`OffsetNumberNext(BloomPageGetMaxOffset(page))));`
`110`	`110`
`111`		`-if (BloomPageGetFreeSpace(&state,page)>state.sizeOfBloomTuple&&`
	`111`	`+/*`
	`112`	`+ * Add page to notFullPage list if we will not mark page as deleted and`
	`113`	`+ * there is a free space on it`
	`114`	`+ */`
	`115`	`+if (BloomPageGetMaxOffset(page)==0&&`
	`116`	`+BloomPageGetFreeSpace(&state,page)>state.sizeOfBloomTuple&&`
`112`	`117`	`countPage<BloomMetaBlockN)`
`113`	`118`	`notFullPage[countPage++]=blkno;`
`114`	`119`
`115`	`120`	`/* Did we delete something? */`
`116`	`121`	`if (itupPtr!=itup)`
`117`	`122`	`{`
`118`	`123`	`/* Is it empty page now? */`
`119`		`-if (itupPtr==BloomPageGetData(page))`
	`124`	`+if (BloomPageGetMaxOffset(page)==0)`
`120`	`125`	`BloomPageSetDeleted(page);`
`121`	`126`	`/* Adjust pg_lower */`
`122`	`127`	`((PageHeader)page)->pd_lower= (Pointer)itupPtr-page;`

Comments

(0)