Fix checking for recovery state in WaitForLSN()

akorotkov · akorotkov · commit 7742f99a02ed · 2025-11-07T23:34:50.000+02:00
We only need to do it for WAIT_LSN_TYPE_REPLAY.  WAIT_LSN_TYPE_FLUSH can work
for both primary and follower.
diff --git a/src/backend/access/transam/xlogwait.c b/src/backend/access/transam/xlogwait.c
@@ -347,7 +347,7 @@ WaitForLSN(WaitLSNType lsnType, XLogRecPtr targetLSN, int64 timeout)
 			currentLSN = GetFlushRecPtr(NULL);
 
 		/* Check that recovery is still in-progress */
-		if (!RecoveryInProgress())
+		if (lsnType == WAIT_LSN_TYPE_REPLAY && !RecoveryInProgress())
 		{
 			/*
 			 * Recovery was ended, but check if target LSN was already

Original file line number	Diff line number	Diff line change
`@@ -347,7 +347,7 @@ WaitForLSN(WaitLSNType lsnType, XLogRecPtr targetLSN, int64 timeout)`
`347`	`347`	`currentLSN = GetFlushRecPtr(NULL);`
`348`	`348`
`349`	`349`	`/* Check that recovery is still in-progress */`
`350`		`- if (!RecoveryInProgress())`
	`350`	`+ if (lsnType == WAIT_LSN_TYPE_REPLAY && !RecoveryInProgress())`
`351`	`351`	`{`
`352`	`352`	`/*`
`353`	`353`	`* Recovery was ended, but check if target LSN was already`