NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit57b100f

committed

Fix some more bugs in GIN's WAL replay logic.

In commit4016bde I fixed a bunch ofginxlog.c bugs having to do with not handling XLogReadBuffer failurescorrectly. However, in ginRedoUpdateMetapage and ginRedoDeleteListPages,I unaccountably thought that failure to read the metapage would beimpossible and just put in an elog(PANIC) call. This is of course wrong:failure is exactly what will happen if the index got dropped (or rebuilt)between creation of the WAL record and the crash we're trying to recoverfrom. I believe this explains Nicholas Wilson's recent report of theseerrors getting reached.Also, fix memory leak in forgetIncompleteSplit. This wasn't of muchconcern when the code was written, but in a long-running standby serverpage split records could be expected to accumulate indefinitely.Back-patch to 8.4 --- before that, GIN didn't have a metapage.

1 parent64c47e4 commit57b100fCopy full SHA for 57b100f

File tree

1 file changed

-3

lines changed

src/backend/access/gin
- ginxlog.c

1 file changed

-3

lines changed

`‎src/backend/access/gin/ginxlog.c`

Lines changed: 6 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -59,9 +59,12 @@ forgetIncompleteSplit(RelFileNode node, BlockNumber leftBlkno, BlockNumber updat`
`59`	`59`	`{`
`60`	`60`	`ginIncompleteSplitsplit= (ginIncompleteSplit)lfirst(l);`
`61`	`61`
`62`		`-if (RelFileNodeEquals(node,split->node)&&leftBlkno==split->leftBlkno&&updateBlkno==split->rightBlkno)`
	`62`	`+if (RelFileNodeEquals(node,split->node)&&`
	`63`	`+leftBlkno==split->leftBlkno&&`
	`64`	`+updateBlkno==split->rightBlkno)`
`63`	`65`	`{`
`64`	`66`	`incomplete_splits=list_delete_ptr(incomplete_splits,split);`
	`67`	`+pfree(split);`
`65`	`68`	`break;`
`66`	`69`	`}`
`67`	`70`	`}`
`@@ -487,7 +490,7 @@ ginRedoUpdateMetapage(XLogRecPtr lsn, XLogRecord *record)`
`487`	`490`
`488`	`491`	`metabuffer=XLogReadBuffer(data->node,GIN_METAPAGE_BLKNO, false);`
`489`	`492`	`if (!BufferIsValid(metabuffer))`
`490`		`-elog(PANIC,"GIN metapage disappeared");`
	`493`	`+return;/* assume index was deleted, nothing to do */`
`491`	`494`	`metapage=BufferGetPage(metabuffer);`
`492`	`495`
`493`	`496`	`if (!XLByteLE(lsn,PageGetLSN(metapage)))`
`@@ -632,7 +635,7 @@ ginRedoDeleteListPages(XLogRecPtr lsn, XLogRecord *record)`
`632`	`635`
`633`	`636`	`metabuffer=XLogReadBuffer(data->node,GIN_METAPAGE_BLKNO, false);`
`634`	`637`	`if (!BufferIsValid(metabuffer))`
`635`		`-elog(PANIC,"GIN metapage disappeared");`
	`638`	`+return;/* assume index was deleted, nothing to do */`
`636`	`639`	`metapage=BufferGetPage(metabuffer);`
`637`	`640`
`638`	`641`	`if (!XLByteLE(lsn,PageGetLSN(metapage)))`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit57b100f

File tree

1 file changed

1 file changed

`‎src/backend/access/gin/ginxlog.c`

0 commit comments