From: Peter Eisentraut <peter_e@gmx.net>
Date: Tue, 15 Aug 2017 19:13:06 +0000 (-0400)
Subject: Simplify some code in logical replication launcher
X-Git-Tag: REL_10_BETA4~45
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=292d9b66e2251df4767a43d78927a614561a21d6;p=thirdparty%2Fpostgresql.git

Simplify some code in logical replication launcher

Avoid unnecessary locking calls when a subscription is disabled.

Author: Yugo Nagata <nagata@sraoss.co.jp>
---

diff --git a/src/backend/replication/logical/launcher.c b/src/backend/replication/logical/launcher.c
index 4c6d4b27723..6c894421a39 100644
--- a/src/backend/replication/logical/launcher.c
+++ b/src/backend/replication/logical/launcher.c
@@ -929,11 +929,14 @@ ApplyLauncherMain(Datum main_arg)
 				Subscription *sub = (Subscription *) lfirst(lc);
 				LogicalRepWorker *w;
 
+				if (!sub->enabled)
+					continue;
+
 				LWLockAcquire(LogicalRepWorkerLock, LW_SHARED);
 				w = logicalrep_worker_find(sub->oid, InvalidOid, false);
 				LWLockRelease(LogicalRepWorkerLock);
 
-				if (sub->enabled && w == NULL)
+				if (w == NULL)
 				{
 					last_start_time = now;
 					wait_time = wal_retrieve_retry_interval;