From: Tom Lane Date: Sun, 23 Feb 2003 04:48:38 +0000 (+0000) Subject: If a shutdown request comes in while we're still starting up, don't X-Git-Tag: REL7_3_3~75 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=1f3e24dee7e5dad3d6e34871ba78b7cb59a20771;p=postgresql If a shutdown request comes in while we're still starting up, don't service it until after we execute SetThisStartUpID(). Else shutdown process will write the wrong SUI into the shutdown checkpoint, which seems likely to be trouble --- although I've not quite figured out how significant it really is. --- diff --git a/src/backend/postmaster/postmaster.c b/src/backend/postmaster/postmaster.c index d8b36b37cb..8f72ddc1ab 100644 --- a/src/backend/postmaster/postmaster.c +++ b/src/backend/postmaster/postmaster.c @@ -37,7 +37,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/postmaster/postmaster.c,v 1.290.2.2 2003/01/16 00:27:17 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/postmaster/postmaster.c,v 1.290.2.3 2003/02/23 04:48:38 tgl Exp $ * * NOTES * @@ -1621,7 +1621,26 @@ reaper(SIGNAL_ARGS) ExitPostmaster(1); } StartupPID = 0; + + /* + * Startup succeeded - remember its ID and RedoRecPtr. + * + * NB: this MUST happen before we fork a checkpoint or shutdown + * subprocess, else they will have wrong local ThisStartUpId. + */ + SetThisStartUpID(); + FatalError = false; /* done with recovery */ + + /* + * Arrange for first checkpoint to occur after standard delay. + */ + CheckPointPID = 0; + checkpointed = time(NULL); + + /* + * Go to shutdown mode if a shutdown request was pending. + */ if (Shutdown > NoShutdown) { if (ShutdownPID > 0) @@ -1633,17 +1652,6 @@ reaper(SIGNAL_ARGS) ShutdownPID = ShutdownDataBase(); } - /* - * Startup succeeded - remember its ID and RedoRecPtr - */ - SetThisStartUpID(); - - /* - * Arrange for first checkpoint to occur after standard delay. - */ - CheckPointPID = 0; - checkpointed = time(NULL); - goto reaper_done; }