Shut down WAL receiver if it's still running at end of recovery. We used to

just check that it's not running and PANIC if it was, but that can rightfully
happen if recovery stops at recovery target.

diff --git a/src/backend/access/transam/xlog.c b/src/backend/access/transam/xlog.c
index b0e4c41d6f1c8cc66889967743174e527bba2360..e71090f71b5e8441aa06cd5b0a8368d87293f142 100644 (file)
--- a/src/backend/access/transam/xlog.c
+++ b/src/backend/access/transam/xlog.c
@@ -6611,12 +6611,11 @@ StartupXLOG(void)
        }
 
        /*
-        * If we launched a WAL receiver, it should be gone by now. It will trump
-        * over the startup checkpoint and subsequent records if it's still alive,
-        * so be extra sure that it's gone.
+        * Kill WAL receiver, if it's still running, before we continue to write
+        * the startup checkpoint record. It will trump over the checkpoint and
+        * subsequent records if it's still alive when we start writing WAL.
         */
-       if (WalRcvInProgress())
-               elog(PANIC, "wal receiver still active");
+       ShutdownWalRcv();
 
        /*
         * We don't need the latch anymore. It's not strictly necessary to disown