improve the entries list' sweep when the max usage count is changed

Signed-off-by: Ludovic Orban <lorban@bitronix.be>

jetty-util/src/main/java/org/eclipse/jetty/util/Pool.java

@@ -29,6 +29,7 @@
 import java.util.concurrent.ThreadLocalRandom;
 import java.util.concurrent.atomic.AtomicInteger;
 import java.util.function.Function;
+import java.util.stream.Collectors;
 
 import org.eclipse.jetty.util.component.Dumpable;
 import org.eclipse.jetty.util.component.DumpableCollection;
@@ -190,18 +191,18 @@ public final void setMaxUsageCount(int maxUsageCount)
         if (closed)
             return;
 
-        List<Entry> copy;
+        // Iterate the entries, remove overused ones and collect a list of the closeable removed ones.
+        List<Closeable> copy;
         try (Locker.Lock l = locker.lock())
         {
-            copy = new ArrayList<>(entries);
+            copy = entries.stream()
+                .filter(entry -> entry.isIdleAndOverUsed() && remove(entry) && entry.pooled instanceof Closeable)
+                .map(entry -> (Closeable)entry.pooled)
+                .collect(Collectors.toList());
         }
 
-        // Iterate the copy and close overused entries.
-        for (Entry entry : copy)
-        {
-            if (entry.isIdleAndOverUsed() && remove(entry) && entry.pooled instanceof Closeable)
-                IO.close((Closeable)entry.pooled);
-        }
+        // Iterate the copy and close the collected entries.
+        copy.forEach(IO::close);
     }
 
     /**