the integration tests are timing out because the following hangs: org.apache.geode.cache.client.internal.ConnectionPoolImplJUnitTest.testExecuteOp() This test creates a mocked Op whose attempt method always throws: SocketTimeoutException. org.apache.geode.cache.client.internal.OpExecutorImpl.execute(Op, int) has some exception handling that end up calling exchangeConnection and your new implementation of it ends up always creating a connection. So we retry and keep getting the SocketTimeoutException. The retry logic keeps adding the server that failed to a set of servers to ignore. So the old exchange would eventually throw NoAvailableServersException once all the servers were added to the attemptedServers set.
[ Full content available at: https://github.com/apache/geode/pull/3296 ] This message was relayed via gitbox.apache.org for [email protected]
