Move elog(DEBUG4) call outside the locked area, per suggestion from Tom Lane.

diff --git a/src/backend/storage/ipc/sinvaladt.c b/src/backend/storage/ipc/sinvaladt.c
index 485506a162d0cccce2b680a485cfc384d803f85a..99c5222e231c9f7f742b5c8c984f2b78e827eb1f 100644 (file)
--- a/src/backend/storage/ipc/sinvaladt.c
+++ b/src/backend/storage/ipc/sinvaladt.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/backend/storage/ipc/sinvaladt.c,v 1.68 2008/03/17 11:50:27 alvherre Exp $
+ *       $PostgreSQL: pgsql/src/backend/storage/ipc/sinvaladt.c,v 1.69 2008/03/18 12:36:43 alvherre Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -330,12 +330,8 @@ SIInsertDataEntry(SharedInvalidationMessage *data)
         * queries, but if a backend is sitting idle then it won't be starting
         * transactions and so won't be reading SI entries.
         */
-       if (numMsgs == (MAXNUMMESSAGES * 70 / 100) &&
-               IsUnderPostmaster)
-       {
-               elog(DEBUG4, "SI table is 70%% full, signaling postmaster");
+       if (numMsgs == (MAXNUMMESSAGES * 70 / 100) && IsUnderPostmaster)
                signal_postmaster = true;
-       }
 
        /*
         * Insert new message into proper slot of circular buffer
@@ -346,7 +342,10 @@ SIInsertDataEntry(SharedInvalidationMessage *data)
        LWLockRelease(SInvalLock);
 
        if (signal_postmaster)
+       {
+               elog(DEBUG4, "SI table is 70%% full, signaling postmaster");
                SendPostmasterSignal(PMSIGNAL_WAKEN_CHILDREN);
+       }
 
        return true;
 }