Redis Hostname provider - do not call getHosts() twice

quarkusio · Jun 4, 2023 · 06b69ea · 06b69ea
1 parent 947d3a9
commit 06b69ea
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/...client/runtime/src/main/java/io/quarkus/redis/runtime/client/VertxRedisClientFactory.java b/...client/runtime/src/main/java/io/quarkus/redis/runtime/client/VertxRedisClientFactory.java
@@ -10,6 +10,7 @@
 import java.net.URI;
 import java.util.ArrayList;
 import java.util.List;
+import java.util.Set;
 
 import io.quarkus.arc.Arc;
 import io.quarkus.arc.ArcContainer;
@@ -51,8 +52,9 @@ public static Redis create(String name, Vertx vertx, RedisClientConfig config) {
             }
         } else if (config.hostsProviderName.isPresent()) {
             RedisHostsProvider hostsProvider = findProvider(config.hostsProviderName.get());
-            hosts.addAll(hostsProvider.getHosts());
-            for (URI uri : hostsProvider.getHosts()) {
+            Set<URI> computedHosts = hostsProvider.getHosts();
+            hosts.addAll(computedHosts);
+            for (URI uri : computedHosts) {
                 options.addConnectionString(uri.toString());
             }
         } else {