Slight refactor for ExecOpenScanRelation(): we can use

ExecRelationIsTargetRelation() to check if the relation is a target
rel, rather than scanning through the result relation array ourselves.

diff --git a/src/backend/executor/execUtils.c b/src/backend/executor/execUtils.c
index 90d0f4236087f2eb2024ca68d20ebfc02ff03dd8..016cc7e6883ccef3c2fec23eb791e39c15e9dbee 100644 (file)
--- a/src/backend/executor/execUtils.c
+++ b/src/backend/executor/execUtils.c
@@ -814,24 +814,16 @@ ExecOpenScanRelation(EState *estate, Index scanrelid)
 {
        Oid                     reloid;
        LOCKMODE        lockmode;
-       ResultRelInfo *resultRelInfos;
-       int                     i;
 
        /*
-        * First determine the lock type we need.  Scan to see if target relation
-        * is either a result relation or a FOR UPDATE/FOR SHARE relation.
+        * Determine the lock type we need.  First, scan to see if target
+        * relation is a result relation.
         */
        lockmode = AccessShareLock;
-       resultRelInfos = estate->es_result_relations;
-       for (i = 0; i < estate->es_num_result_relations; i++)
-       {
-               if (resultRelInfos[i].ri_RangeTableIndex == scanrelid)
-               {
-                       lockmode = NoLock;
-                       break;
-               }
-       }
+       if (ExecRelationIsTargetRelation(estate, scanrelid))
+               lockmode = NoLock;
 
+       /* If not, check if it's a FOR UPDATE/FOR SHARE relation */
        if (lockmode == AccessShareLock)
        {
                ListCell   *l;
@@ -850,7 +842,6 @@ ExecOpenScanRelation(EState *estate, Index scanrelid)
 
        /* OK, open the relation and acquire lock as needed */
        reloid = getrelid(scanrelid, estate->es_range_table);
-
        return heap_open(reloid, lockmode);
 }