diff --git a/src/backend/access/gin/ginxlog.c b/src/backend/access/gin/ginxlog.c index b84ecf2ab1..f3c82ac836 100644 --- a/src/backend/access/gin/ginxlog.c +++ b/src/backend/access/gin/ginxlog.c @@ -531,7 +531,24 @@ ginRedoDeletePage(XLogReaderState *record) page = BufferGetPage(dbuffer); Assert(GinPageIsData(page)); GinPageGetOpaque(page)->flags = GIN_DELETED; - GinPageSetDeleteXid(page, data->deleteXid); + + /* + * deleteXid field of ginxlogDeletePage was added during backpatching. + * But, non-backpatched instances will continue generate WAL without + * this field. We should be able to correctly apply that. We can + * distinguish new WAL records by size their data, because + * ginxlogDeletePage changes its size on both 32-bit and 64-bit + * platforms. + */ + StaticAssertStmt(sizeof(ginxlogDeletePage) != + sizeof(ginxlogDeletePageOld), + "ginxlogDeletePage size should be changed " + "with addition of deleteXid field"); + Assert(XLogRecGetDataLen(record) == sizeof(ginxlogDeletePage) || + XLogRecGetDataLen(record) == sizeof(ginxlogDeletePageOld)); + if (XLogRecGetDataLen(record) == sizeof(ginxlogDeletePage)) + GinPageSetDeleteXid(page, data->deleteXid); + PageSetLSN(page, lsn); MarkBufferDirty(dbuffer); } diff --git a/src/include/access/ginxlog.h b/src/include/access/ginxlog.h index b2f3126aa8..87b1e7bbda 100644 --- a/src/include/access/ginxlog.h +++ b/src/include/access/ginxlog.h @@ -161,6 +161,16 @@ typedef struct ginxlogDeletePage TransactionId deleteXid; /* last Xid which could see this page in scan */ } ginxlogDeletePage; +/* + * Previous version of ginxlogDeletePage struct, which didn't have deleteXid + * field. Used for size comparison (see ginRedoDeletePage()). + */ +typedef struct ginxlogDeletePageOld +{ + OffsetNumber parentOffset; + BlockNumber rightLink; +} ginxlogDeletePageOld; + #define XLOG_GIN_UPDATE_META_PAGE 0x60 /*