Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

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

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit9bc70b1

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 wehave all the information required to re-initialize in the WAL recordanyway. Before this fix, a full-page image was always generated, unlessfull_page_writes=off, because when the page is initialized its LSN isalways 0. In stable-branches, keep the code to restore the backup blocksif they exist, in case that the WAL is generated with an older minorversion, but in master Assert that there are no full-page images.In the redo routine, add missing "off++". Otherwise the tuples are addedto the page in reverse order. That happens to be harmless because wealways 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, becausethe page was always restored from a full-page image.Backpatch to all supported versions.
1 parent70e7be2 commit9bc70b1

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

‎src/backend/access/gin/ginfast.c‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -120,8 +120,7 @@ writeListPage(Relation index, Buffer buffer,
120120
rdata[0].len=sizeof(ginxlogInsertListPage);
121121
rdata[0].next=rdata+1;
122122

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

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

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -621,7 +621,11 @@ ginRedoInsertListPage(XLogRecPtr lsn, XLogRecord *record)
621621
tupsize;
622622
IndexTupletuples= (IndexTuple) (XLogRecGetData(record)+sizeof(ginxlogInsertListPage));
623623

624-
/* If we have a full-page image, restore it and we're done */
624+
/*
625+
* If we have a full-page image, restore it and we're done. (As the code
626+
* stands, we never create full-page images, but we used to. Cope if
627+
* we're reading WAL generated with an older minor version.)
628+
*/
625629
if (record->xl_info&XLR_BKP_BLOCK(0))
626630
{
627631
(void)RestoreBackupBlock(lsn,record,0, false, false);
@@ -655,6 +659,7 @@ ginRedoInsertListPage(XLogRecPtr lsn, XLogRecord *record)
655659
elog(ERROR,"failed to add item to index page");
656660

657661
tuples= (IndexTuple) (((char*)tuples)+tupsize);
662+
off++;
658663
}
659664

660665
PageSetLSN(page,lsn);

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp