Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit d272244

Browse files
committed
Fix two bugs in WAL-logging of GIN pending-list pages.
In writeListPage, never take a full-page image of the page, because we have all the information required to re-initialize in the WAL record anyway. Before this fix, a full-page image was always generated, unless full_page_writes=off, because when the page is initialized its LSN is always 0. In stable-branches, keep the code to restore the backup blocks if they exist, in case that the WAL is generated with an older minor version, but in master Assert that there are no full-page images. In the redo routine, add missing "off++". Otherwise the tuples are added to the page in reverse order. That happens to be harmless because we always scan and remove all the tuples together, but it was clearly wrong. Also, it was masked by the first bug unless full_page_writes=off, because the page was always restored from a full-page image. Backpatch to all supported versions.
1 parent 728c06f commit d272244

File tree

2 files changed

+6
-8
lines changed

2 files changed

+6
-8
lines changed

src/backend/access/gin/ginfast.c

+1-2
Original file line numberDiff line numberDiff line change
@@ -119,8 +119,7 @@ writeListPage(Relation index, Buffer buffer,
119119
rdata[0].len = sizeof(ginxlogInsertListPage);
120120
rdata[0].next = rdata + 1;
121121

122-
rdata[1].buffer = buffer;
123-
rdata[1].buffer_std = true;
122+
rdata[1].buffer = InvalidBuffer;
124123
rdata[1].data = workspace;
125124
rdata[1].len = size;
126125
rdata[1].next = NULL;

src/backend/access/gin/ginxlog.c

+5-6
Original file line numberDiff line numberDiff line change
@@ -814,12 +814,10 @@ ginRedoInsertListPage(XLogRecPtr lsn, XLogRecord *record)
814814
tupsize;
815815
IndexTuple tuples = (IndexTuple) (XLogRecGetData(record) + sizeof(ginxlogInsertListPage));
816816

817-
/* If we have a full-page image, restore it and we're done */
818-
if (record->xl_info & XLR_BKP_BLOCK(0))
819-
{
820-
(void) RestoreBackupBlock(lsn, record, 0, false, false);
821-
return;
822-
}
817+
/*
818+
* Backup blocks are not used, we always re-initialize the page.
819+
*/
820+
Assert(!(record->xl_info & XLR_BKP_BLOCK_MASK));
823821

824822
buffer = XLogReadBuffer(data->node, data->blkno, true);
825823
Assert(BufferIsValid(buffer));
@@ -848,6 +846,7 @@ ginRedoInsertListPage(XLogRecPtr lsn, XLogRecord *record)
848846
elog(ERROR, "failed to add item to index page");
849847

850848
tuples = (IndexTuple) (((char *) tuples) + tupsize);
849+
off++;
851850
}
852851

853852
PageSetLSN(page, lsn);

0 commit comments

Comments
 (0)