Now that availableConnections is a concurrent queue and other threads can be concurrently adding and removing from it, it seems wrong to have the size of the queue at one point in time limit our iterations. The other complication is that we add items we took off the head back to the tail in this loop so we could actually see them more than once. We should rethink this code now that it no longer holds a lock during the iteration.
[ Full content available at: https://github.com/apache/geode/pull/3296 ] This message was relayed via gitbox.apache.org for [email protected]
