v3-0001-Keep-the-return-value-of-XLogInsertRecord-for-XLO.patch

application/octet-stream

Filename: v3-0001-Keep-the-return-value-of-XLogInsertRecord-for-XLO.patch
Type: application/octet-stream
Part: 0
Message: Re: Return value of XLogInsertRecord() for XLOG_SWITCH record

Patch

Same data as JSON: GET /api/v1/attachments/:id/patch the parsed metadata as JSON — format, series position, per-file stats; never the diff bytes. API reference →
Format: format-patch
Series: patch v3-0001
Subject: Keep the return value of XLogInsertRecord() for XLOG_SWITCH record consistent with other records patch v3
File+
src/backend/access/transam/xlog.c 8 2
From f362d2df864d18cf6f99a66ce029d7e5c2e033b7 Mon Sep 17 00:00:00 2001
From: "ZizhuanLiu(X-MAN)" <44973863@qq.com>
Date: Mon, 15 Jun 2026 13:15:38 +0800
Subject: [PATCH v3] Keep the return value of XLogInsertRecord() for
 XLOG_SWITCH record consistent with other records patch v3

---
 src/backend/access/transam/xlog.c | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/src/backend/access/transam/xlog.c b/src/backend/access/transam/xlog.c
index d69d03b..cc79df2 100644
--- a/src/backend/access/transam/xlog.c
+++ b/src/backend/access/transam/xlog.c
@@ -1027,8 +1027,14 @@ XLogInsertRecord(XLogRecData *rdata,
 		 */
 		if (inserted)
 		{
-			EndPos = StartPos + SizeOfXLogRecord;
-			if (StartPos / XLOG_BLCKSZ != EndPos / XLOG_BLCKSZ)
+			EndPos = StartPos + MAXALIGN(SizeOfXLogRecord);
+
+			/*
+			 * If the XLOG_SWITCH record crosses a page boundary and actually has
+			 * a part on the next page, we need to consider the page header. This
+			 * is consistent with XLogBytePosToEndRecPtr().
+			 */
+			if (StartPos / XLOG_BLCKSZ != (EndPos - 1) / XLOG_BLCKSZ)
 			{
 				uint64		offset = XLogSegmentOffset(EndPos, wal_segment_size);
 
-- 
2.43.0