From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Tue, 31 Jul 2007 16:36:07 +0000 (+0000)
Subject: If we're gonna use ExecRelationIsTargetRelation here, might as well
X-Git-Tag: REL9_0_0~5247
X-Git-Url: http://git.osdn.net/view?a=commitdiff_plain;h=bc421c35b1ee6beada9b357429847a2b3a0e4449;p=pg-rex%2Fsyncrep.git

If we're gonna use ExecRelationIsTargetRelation here, might as well
simplify a bit further.
---

diff --git a/src/backend/executor/execUtils.c b/src/backend/executor/execUtils.c
index 91baffb4e7..fe76a33774 100644
--- a/src/backend/executor/execUtils.c
+++ b/src/backend/executor/execUtils.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/executor/execUtils.c,v 1.148 2007/07/27 19:09:04 neilc Exp $
+ *	  $PostgreSQL: pgsql/src/backend/executor/execUtils.c,v 1.149 2007/07/31 16:36:07 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -816,15 +816,14 @@ ExecOpenScanRelation(EState *estate, Index scanrelid)
 	LOCKMODE	lockmode;
 
 	/*
-	 * Determine the lock type we need.  First, scan to see if target
-	 * relation is a result relation.
+	 * Determine the lock type we need.  First, scan to see if target relation
+	 * is a result relation.  If not, check if it's a FOR UPDATE/FOR SHARE
+	 * relation.  In either of those cases, we got the lock already.
 	 */
 	lockmode = AccessShareLock;
 	if (ExecRelationIsTargetRelation(estate, scanrelid))
 		lockmode = NoLock;
-
-	/* If not, check if it's a FOR UPDATE/FOR SHARE relation */
-	if (lockmode == AccessShareLock)
+	else
 	{
 		ListCell   *l;