Commitfde7172

committed

Fix setting of right bound at GIN page split.

Broken by my refactoring.

1 parent8dace66 commitfde7172Copy full SHA for fde7172

File tree

-7

lines changed

-7

lines changed

Lines changed: 7 additions & 7 deletions

Original file line number	Diff line number	Diff line change
`@@ -548,7 +548,12 @@ dataSplitPage(GinBtree btree, Buffer lbuf, Buffer rbuf, OffsetNumber off, XLogRe`
`548`	`548`
`549`	`549`	`/* set up right bound for left page */`
`550`	`550`	`bound=GinDataPageGetRightBound(lpage);`
`551`		`-*bound=btree->pitem.key;`
	`551`	`+if (GinPageIsLeaf(lpage))`
	`552`	`+bound=GinDataPageGetItemPointer(lpage,`
	`553`	`+GinPageGetOpaque(lpage)->maxoff);`
	`554`	`+else`
	`555`	`+*bound=GinDataPageGetPostingItem(lpage,`
	`556`	`+GinPageGetOpaque(lpage)->maxoff)->key;`
`552`	`557`
`553`	`558`	`/* set up right bound for right page */`
`554`	`559`	`bound=GinDataPageGetRightBound(rpage);`
`@@ -577,12 +582,7 @@ dataSplitPage(GinBtree btree, Buffer lbuf, Buffer rbuf, OffsetNumber off, XLogRe`
`577`	`582`
`578`	`583`	`/* Prepare a downlink tuple for insertion to the parent */`
`579`	`584`	`PostingItemSetBlockNumber(&(btree->pitem),BufferGetBlockNumber(lbuf));`
`580`		`-if (GinPageIsLeaf(lpage))`
`581`		`-btree->pitem.key=*GinDataPageGetItemPointer(lpage,`
`582`		`-GinPageGetOpaque(lpage)->maxoff);`
`583`		`-else`
`584`		`-btree->pitem.key=GinDataPageGetPostingItem(lpage,`
`585`		`-GinPageGetOpaque(lpage)->maxoff)->key;`
	`585`	`+btree->pitem.key=*GinDataPageGetRightBound(lpage);`
`586`	`586`	`btree->rightblkno=BufferGetBlockNumber(rbuf);`
`587`	`587`
`588`	`588`	`returnlpage;`

Comments

(0)