Issue a warning if a change-on-restart-only postgresql.conf value is

modified  and the server config files are reloaded

diff --git a/doc/TODO b/doc/TODO
index d7e210651fc46aa7639d79396430235f410ee230..b38a717f1e24a1149ac3f71ff6402a667882b431 100644 (file)
--- a/doc/TODO
+++ b/doc/TODO
@@ -83,7 +83,7 @@ Administration
    o %Allow postgresql.conf file values to be changed via an SQL
      API, perhaps using SET GLOBAL
    o Allow the server to be stopped/restarted via an SQL API
-   o Issue a warning if a change-on-restart-only postgresql.conf value
+   o -Issue a warning if a change-on-restart-only postgresql.conf value
      is modified  and the server config files are reloaded
    o Mark change-on-restart-only values in postgresql.conf
 

diff --git a/doc/src/FAQ/TODO.html b/doc/src/FAQ/TODO.html
index 9aafe3e9c305aa7faf81adc076b02eab67c04eaf..d9d21ac4c74170471b2db43498ba033dab63e295 100644 (file)
--- a/doc/src/FAQ/TODO.html
+++ b/doc/src/FAQ/TODO.html
@@ -26,7 +26,7 @@ first.
 
 <ul>
   <li>%Remove behavior of postmaster -o
-  </li><li>-*%Allow pooled connections to list all prepared statements*
+  </li><li>-<em>%Allow pooled connections to list all prepared statements</em>
 <p>  This would allow an application inheriting a pooled connection to know
   the statements prepared in the current session.
 </p>
@@ -79,8 +79,8 @@ first.
     </li><li>%Allow postgresql.conf file values to be changed via an SQL
           API, perhaps using SET GLOBAL
     </li><li>Allow the server to be stopped/restarted via an SQL API
-    </li><li>Issue a warning if a change-on-restart-only postgresql.conf value
-          is modified  and the server config files are reloaded
+    </li><li>-<em>Issue a warning if a change-on-restart-only postgresql.conf value
+          is modified  and the server config files are reloaded</em>
     </li><li>Mark change-on-restart-only values in postgresql.conf
   </li></ul>
   </li><li>Tablespaces

diff --git a/src/backend/utils/misc/guc.c b/src/backend/utils/misc/guc.c
index e16bbd057b30772d3dc42a818258872321fa2136..6ff6382a482c93ac903771e9b75ee01be638993f 100644 (file)
--- a/src/backend/utils/misc/guc.c
+++ b/src/backend/utils/misc/guc.c
@@ -10,7 +10,7 @@
  * Written by Peter Eisentraut <peter_e@gmx.net>.
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/backend/utils/misc/guc.c,v 1.309 2006/01/09 10:05:31 petere Exp $
+ *   $PostgreSQL: pgsql/src/backend/utils/misc/guc.c,v 1.310 2006/02/04 12:50:47 petere Exp $
  *
  *--------------------------------------------------------------------
  */
@@ -2201,6 +2201,7 @@ static void ReportGUCOption(struct config_generic * record);
 static void ShowGUCConfigOption(const char *name, DestReceiver *dest);
 static void ShowAllGUCConfig(DestReceiver *dest);
 static char *_ShowOption(struct config_generic * record);
+static bool is_newvalue_equal(struct config_generic *record, const char *newvalue);
 
 
 /*
@@ -3631,7 +3632,15 @@ set_config_option(const char *name, const char *value,
            break;
        case PGC_POSTMASTER:
            if (context == PGC_SIGHUP)
+           {
+               if (changeVal && !is_newvalue_equal(record, value))
+                   ereport(elevel,
+                           (errcode(ERRCODE_CANT_CHANGE_RUNTIME_PARAM),
+                            errmsg("parameter \"%s\" cannot be changed after server start; configuration file change ignored",
+                                   name)));
+
                return true;
+           }
            if (context != PGC_POSTMASTER)
            {
                ereport(elevel,
@@ -5079,6 +5088,44 @@ _ShowOption(struct config_generic * record)
 }
 
 
+static bool
+is_newvalue_equal(struct config_generic *record, const char *newvalue)
+{
+   switch (record->vartype)
+   {
+       case PGC_BOOL:
+       {
+           struct config_bool *conf = (struct config_bool *) record;
+           bool newval;
+
+           return parse_bool(newvalue, &newval) && *conf->variable == newval;
+       }
+       case PGC_INT:
+       {
+           struct config_int *conf = (struct config_int *) record;
+           int newval;
+
+           return parse_int(newvalue, &newval) && *conf->variable == newval;
+       }
+       case PGC_REAL:
+       {
+           struct config_real *conf = (struct config_real *) record;
+           double newval;
+
+           return parse_real(newvalue, &newval) && *conf->variable == newval;
+       }
+       case PGC_STRING:
+       {
+           struct config_string *conf = (struct config_string *) record;
+
+           return strcmp(*conf->variable, newvalue) == 0;
+       }
+   }
+
+   return false;
+}
+
+
 #ifdef EXEC_BACKEND
 
 /*