Replace for-loop with explicit at-most-once iteration

knutwalker · Mats-SX · vnickolov · commit 8b4caac9c6d4 · 2021-05-20T16:40:39.000+01:00
Co-Authored-By: Mats Rydberg &lt;mats@neotechnology.com&gt;
diff --git a/core/src/main/java/org/neo4j/graphalgo/core/loading/GraphStoreCatalog.java b/core/src/main/java/org/neo4j/graphalgo/core/loading/GraphStoreCatalog.java
@@ -120,7 +120,8 @@ public static void remove(
             ));
         }
 
-        for (var username: usersWithMatchingGraphs) {
+        if (!usersWithMatchingGraphs.isEmpty()) {
+            var username = usersWithMatchingGraphs.iterator().next();
             getUserCatalog(username).remove(
                 userCatalogKey,
                 removedGraphConsumer,

Original file line number	Diff line number	Diff line change
`@@ -120,7 +120,8 @@ public static void remove(`
`120`	`120`	`));`
`121`	`121`	`}`
`122`	`122`
`123`		`- for (var username: usersWithMatchingGraphs) {`
	`123`	`+ if (!usersWithMatchingGraphs.isEmpty()) {`
	`124`	`+ var username = usersWithMatchingGraphs.iterator().next();`
`124`	`125`	`getUserCatalog(username).remove(`
`125`	`126`	`userCatalogKey,`
`126`	`127`	`removedGraphConsumer,`