From: Simon Riggs <simon@2ndQuadrant.com>
Date: Mon, 4 Apr 2011 22:23:13 +0000 (+0100)
Subject: Avoid assuming there will be only 3 states for synchronous_commit.
X-Git-Tag: REL9_1_BETA1~158
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=88f32b7ca29982f286b61a9d0dd29be4b8a01c25;p=postgresql

Avoid assuming there will be only 3 states for synchronous_commit.
Also avoid hardcoding the current default state by giving it the name
"on" and replace with a meaningful name that reflects its behaviour.
Coding only, no change in behaviour.
---

diff --git a/src/backend/access/transam/xact.c b/src/backend/access/transam/xact.c
index 503506f1f0..55aee87910 100644
--- a/src/backend/access/transam/xact.c
+++ b/src/backend/access/transam/xact.c
@@ -1056,7 +1056,7 @@ RecordTransactionCommit(void)
 	 * if all to-be-deleted tables are temporary though, since they are lost
 	 * anyway if we crash.)
 	 */
-	if ((wrote_xlog && synchronous_commit >= SYNCHRONOUS_COMMIT_LOCAL) ||
+	if ((wrote_xlog && synchronous_commit > SYNCHRONOUS_COMMIT_OFF) ||
 		forceSyncCommit || nrels > 0)
 	{
 		/*
diff --git a/src/backend/postmaster/autovacuum.c b/src/backend/postmaster/autovacuum.c
index 675bfced63..619facc752 100644
--- a/src/backend/postmaster/autovacuum.c
+++ b/src/backend/postmaster/autovacuum.c
@@ -1531,7 +1531,7 @@ AutoVacWorkerMain(int argc, char *argv[])
 	 * if we are waiting for standbys to connect. This is important to
 	 * ensure we aren't blocked from performing anti-wraparound tasks.
 	 */
-	if (synchronous_commit == SYNCHRONOUS_COMMIT_ON)
+	if (synchronous_commit > SYNCHRONOUS_COMMIT_LOCAL_FLUSH)
 		SetConfigOption("synchronous_commit", "local", PGC_SUSET, PGC_S_OVERRIDE);
 
 	/*
diff --git a/src/backend/utils/misc/guc.c b/src/backend/utils/misc/guc.c
index 1f7a7d24f8..2151fde361 100644
--- a/src/backend/utils/misc/guc.c
+++ b/src/backend/utils/misc/guc.c
@@ -355,7 +355,7 @@ static const struct config_enum_entry constraint_exclusion_options[] = {
  * accept all the likely variants of "on" and "off".
  */
 static const struct config_enum_entry synchronous_commit_options[] = {
-	{"local", SYNCHRONOUS_COMMIT_LOCAL, false},
+	{"local", SYNCHRONOUS_COMMIT_LOCAL_FLUSH, false},
 	{"on", SYNCHRONOUS_COMMIT_ON, false},
 	{"off", SYNCHRONOUS_COMMIT_OFF, false},
 	{"true", SYNCHRONOUS_COMMIT_ON, true},
diff --git a/src/include/access/xact.h b/src/include/access/xact.h
index 4410145344..8b60e8f163 100644
--- a/src/include/access/xact.h
+++ b/src/include/access/xact.h
@@ -54,11 +54,14 @@ extern bool XactDeferrable;
 
 typedef enum
 {
-	SYNCHRONOUS_COMMIT_OFF,		/* asynchronous commit */
-	SYNCHRONOUS_COMMIT_LOCAL,	/* wait for only local flush */
-	SYNCHRONOUS_COMMIT_ON		/* wait for local flush and sync rep */
+	SYNCHRONOUS_COMMIT_OFF,			/* asynchronous commit */
+	SYNCHRONOUS_COMMIT_LOCAL_FLUSH,	/* wait for local flush only */
+	SYNCHRONOUS_COMMIT_REMOTE_FLUSH	/* wait for local and remote flush */
 } SyncCommitLevel;
 
+/* Define the default setting for synchonous_commit */
+#define SYNCHRONOUS_COMMIT_ON 	SYNCHRONOUS_COMMIT_REMOTE_FLUSH
+
 /* Synchronous commit level */
 extern int	synchronous_commit;
 
diff --git a/src/include/replication/syncrep.h b/src/include/replication/syncrep.h
index 696edccb12..9022b96585 100644
--- a/src/include/replication/syncrep.h
+++ b/src/include/replication/syncrep.h
@@ -20,7 +20,7 @@
 #include "utils/guc.h"
 
 #define SyncRepRequested() \
-	(max_wal_senders > 0 && synchronous_commit == SYNCHRONOUS_COMMIT_ON)
+	(max_wal_senders > 0 && synchronous_commit > SYNCHRONOUS_COMMIT_LOCAL_FLUSH)
 
 /* syncRepState */
 #define SYNC_REP_NOT_WAITING		0