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 016cc7e6883ccef3c2fec23eb791e39c15e9dbee..232d9a118421df218dcdeda7c2194795fac84b3b 100644 (file)
--- a/src/backend/executor/execUtils.c
+++ b/src/backend/executor/execUtils.c
@@ -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;