Remove newly added useless assertion check

Coverity complained that my commit 80ba4bb38353 added a dubious coding
for a consistency check that there isn't more than one row for a certain
tgrelid/tgparentid combination.  But we don't check for that explicitly
anywhere else, and if we were to do it, it should be a full
shouldn't-happen elog not just an assert.  It doesn't seem that this is
very important anyway, so remove it.

Discussion: https://postgr.es/m/1337562.1627224583@sss.pgh.pa.us

diff --git a/src/backend/commands/trigger.c b/src/backend/commands/trigger.c
index fc0a4b2fa738d9df80c3ee0229e0a570acc46d67..d8890d2c748a7015ddb49879654e72d12a16eed3 100644 (file)
--- a/src/backend/commands/trigger.c
+++ b/src/backend/commands/trigger.c
@@ -1615,7 +1615,6 @@ renametrig_partition(Relation tgrel, Oid partitionId, Oid parentTriggerOid,
    SysScanDesc tgscan;
    ScanKeyData key;
    HeapTuple   tuple;
-   int         found PG_USED_FOR_ASSERTS_ONLY = 0;
 
    /*
     * Given a relation and the OID of a trigger on parent relation, find the
@@ -1636,8 +1635,6 @@ renametrig_partition(Relation tgrel, Oid partitionId, Oid parentTriggerOid,
        if (tgform->tgparentid != parentTriggerOid)
            continue;           /* not our trigger */
 
-       Assert(found++ <= 0);
-
        partitionRel = table_open(partitionId, NoLock);
 
        /* Rename the trigger on this partition */
@@ -1658,6 +1655,9 @@ renametrig_partition(Relation tgrel, Oid partitionId, Oid parentTriggerOid,
            }
        }
        table_close(partitionRel, NoLock);
+
+       /* There should be at most one matching tuple */
+       break;
    }
    systable_endscan(tgscan);
 }