Further simplify a bit of logic in StartupXLOG().

Commit 7ff23c6 left us with two
identical cases. Collapse them.

Author: Robert Haas <robertmhaas@gmail.com>
Discussion: https://postgr.es/m/CA%2BhUKGJ8NRsqgkZEnsnRc2MFROBV-jCnacbYvtpptK2A9YYp9Q%40mail.gmail.com

Author: macdice

src/backend/access/transam/xlog.c

@@ -7890,28 +7890,22 @@ StartupXLOG(void)
 		 * after we're fully out of recovery mode and already accepting
 		 * queries.
 		 */
-		if (ArchiveRecoveryRequested && IsUnderPostmaster)
+		if (ArchiveRecoveryRequested && IsUnderPostmaster &&
+			LocalPromoteIsTriggered)
 		{
-			if (LocalPromoteIsTriggered)
-			{
-				promoted = true;
+			promoted = true;
 
-				/*
-				 * Insert a special WAL record to mark the end of recovery,
-				 * since we aren't doing a checkpoint. That means that the
-				 * checkpointer process may likely be in the middle of a
-				 * time-smoothed restartpoint and could continue to be for
-				 * minutes after this. That sounds strange, but the effect is
-				 * roughly the same and it would be stranger to try to come
-				 * out of the restartpoint and then checkpoint. We request a
-				 * checkpoint later anyway, just for safety.
-				 */
-				CreateEndOfRecoveryRecord();
-			}
-			else
-				RequestCheckpoint(CHECKPOINT_END_OF_RECOVERY |
-								  CHECKPOINT_IMMEDIATE |
-								  CHECKPOINT_WAIT);
+			/*
+			 * Insert a special WAL record to mark the end of recovery, since
+			 * we aren't doing a checkpoint. That means that the checkpointer
+			 * process may likely be in the middle of a time-smoothed
+			 * restartpoint and could continue to be for minutes after this.
+			 * That sounds strange, but the effect is roughly the same and it
+			 * would be stranger to try to come out of the restartpoint and
+			 * then checkpoint. We request a checkpoint later anyway, just for
+			 * safety.
+			 */
+			CreateEndOfRecoveryRecord();
 		}
 		else
 		{