Commit5b2f2af

committed

nbtree page deletion: Add leaftopparent assertion.

Add documenting assertion. This makes it easier to follow how wemaintain the top parent link in target subtree's half-dead/leaf levelpage.

1 parent3d8d578 commit5b2f2afCopy full SHA for 5b2f2af

File tree

-1

lines changed

-1

lines changed

Lines changed: 3 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -2458,6 +2458,9 @@ _bt_unlink_halfdead_page(Relation rel, Buffer leafbuf, BlockNumber scanblkno,`
`2458`	`2458`	`leaftopparent=InvalidBlockNumber;`
`2459`	`2459`	`}`
`2460`	`2460`
	`2461`	`+/* No leaftopparent for level 0 (leaf page) or level 1 target */`
	`2462`	`+Assert(!BlockNumberIsValid(leaftopparent)\|\|targetlevel>1);`
	`2463`	`+`
`2461`	`2464`	`/*`
`2462`	`2465`	`* And next write-lock the (current) right sibling.`
`2463`	`2466`	`*/`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -818,7 +818,7 @@ btree_xlog_unlink_page(uint8 info, XLogReaderState *record)`
`818`	`818`	`safexid=xlrec->safexid;`
`819`	`819`
`820`	`820`	`/* No leaftopparent for level 0 (leaf page) or level 1 target */`
`821`		`-Assert(xlrec->leaftopparent==InvalidBlockNumber\|\|level>1);`
	`821`	`+Assert(!BlockNumberIsValid(xlrec->leaftopparent)\|\|level>1);`
`822`	`822`
`823`	`823`	`/*`
`824`	`824`	`* In normal operation, we would lock all the pages this WAL record`

Comments

(0)