Don't disable commit_ts in standby if enabled locally
authorAlvaro Herrera <[email protected]>
Fri, 2 Oct 2015 15:49:01 +0000 (12:49 -0300)
committerAlvaro Herrera <[email protected]>
Fri, 2 Oct 2015 15:49:01 +0000 (12:49 -0300)
Bug noticed by Fujii Masao

src/backend/access/transam/commit_ts.c

index 79ca04a6eafd285d44e2db6e6d57fc6a54990ed0..24b8291083546a1a9ca8d51892ba2e2d38e6867d 100644 (file)
@@ -583,14 +583,15 @@ CommitTsParameterChange(bool newvalue, bool oldvalue)
         * pg_control.  If the old value was already set, we already did this, so
         * don't do anything.
         *
-        * If the module is disabled in the master, disable it here too.
+        * If the module is disabled in the master, disable it here too, unless
+        * the module is enabled locally.
         */
        if (newvalue)
        {
                if (!track_commit_timestamp && !oldvalue)
                        ActivateCommitTs();
        }
-       else if (oldvalue)
+       else if (!track_commit_timestamp && oldvalue)
                DeactivateCommitTs(false);
 }