Fix incorrect variable datatype.

MasaoFujii · MasaoFujii · commit a2b94693beb1 · 2020-07-08T21:25:33.000+09:00
Since slot_keep_segs indicates the number of WAL segments not LSN, its datatype should not be XLogRecPtr. Back-patch to v13 where this issue was added. Reported-by: Atsushi Torikoshi Author: Atsushi Torikoshi, tweaked by Fujii Masao Discussion: https://postgr.es/m/ebd0d674f3e050222238a960cac5251a@oss.nttdata.com
diff --git a/src/backend/access/transam/xlog.c b/src/backend/access/transam/xlog.c
@@ -9601,7 +9601,7 @@ KeepLogSeg(XLogRecPtr recptr, XLogSegNo *logSegNo)
 		/* Cap by max_slot_wal_keep_size ... */
 		if (max_slot_wal_keep_size_mb >= 0)
 		{
-			XLogRecPtr	slot_keep_segs;
+			uint64		slot_keep_segs;
 
 			slot_keep_segs =
 				ConvertToXSegs(max_slot_wal_keep_size_mb, wal_segment_size);

Original file line number	Diff line number	Diff line change
`@@ -9601,7 +9601,7 @@ KeepLogSeg(XLogRecPtr recptr, XLogSegNo *logSegNo)`
`9601`	`9601`	`/* Cap by max_slot_wal_keep_size ... */`
`9602`	`9602`	`if (max_slot_wal_keep_size_mb >= 0)`
`9603`	`9603`	`{`
`9604`		`- XLogRecPtr slot_keep_segs;`
	`9604`	`+ uint64 slot_keep_segs;`
`9605`	`9605`
`9606`	`9606`	`slot_keep_segs =`
`9607`	`9607`	`ConvertToXSegs(max_slot_wal_keep_size_mb, wal_segment_size);`