Fix hot standby bug with GiST scans.

hlinnaka · hlinnaka · commit 02b9fd73ee57 · 2014-04-08T14:51:56.000+03:00
Don't reset the rightlink of a page when replaying a page update record.
This was a leftover from pre-hot standby days, when it was not possible to
have scans concurrent with WAL replay. Resetting the right-link was not
necessary back then either, but it was done for the sake of tidiness. But
with hot standby, it's wrong, because a concurrent scan might still need it.

Backpatch all versions with hot standby, 9.0 and above.
diff --git a/src/backend/access/gist/gistxlog.c b/src/backend/access/gist/gistxlog.c
@@ -183,7 +183,6 @@ gistRedoPageUpdateRecord(XLogRecPtr lsn, XLogRecord *record)
 		GistPageSetLeaf(page);
 	}
 
-	GistPageGetOpaque(page)->rightlink = InvalidBlockNumber;
 	PageSetLSN(page, lsn);
 	PageSetTLI(page, ThisTimeLineID);
 	MarkBufferDirty(buffer);

Original file line number	Diff line number	Diff line change
`@@ -183,7 +183,6 @@ gistRedoPageUpdateRecord(XLogRecPtr lsn, XLogRecord *record)`
`183`	`183`	`GistPageSetLeaf(page);`
`184`	`184`	`}`
`185`	`185`
`186`		`- GistPageGetOpaque(page)->rightlink = InvalidBlockNumber;`
`187`	`186`	`PageSetLSN(page, lsn);`
`188`	`187`	`PageSetTLI(page, ThisTimeLineID);`
`189`	`188`	`MarkBufferDirty(buffer);`