[GEODE-4178] Failure from ClientServerMiscDUnitTest.testOnSeverMethodsWithCachingProxyClientHA

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.4.0
• Component/s: client/server
• Assignee: Unassigned
• Reporter: Dan Smith
• Created: 02/Jan/18 19:46
• Updated: 01/Feb/18 22:51
• Resolved: 03/Jan/18 22:05

Description

This test failed in a precheckin run:

org.apache.geode.internal.cache.tier.sockets.ClientServerMiscDUnitTest > testOnSeverMethodsWithCachingProxyClientHA FAILED
    org.apache.geode.cache.client.ServerConnectivityException: Pool unexpected socket timed out on client connection=Pooled Connection to cdcef3a9980a:20441: Connection[cdcef3a9980a:20441]@1092101079 attempt=2). Server unreachable: could not connect after 2 attempts
        at org.apache.geode.cache.client.internal.OpExecutorImpl.handleException(OpExecutorImpl.java:786)
        at org.apache.geode.cache.client.internal.OpExecutorImpl.handleException(OpExecutorImpl.java:611)
        at org.apache.geode.cache.client.internal.OpExecutorImpl.execute(OpExecutorImpl.java:175)
        at org.apache.geode.cache.client.internal.OpExecutorImpl.execute(OpExecutorImpl.java:116)
        at org.apache.geode.cache.client.internal.PoolImpl.execute(PoolImpl.java:774)
        at org.apache.geode.cache.client.internal.PutOp.execute(PutOp.java:91)
        at org.apache.geode.cache.client.internal.ServerRegionProxy.put(ServerRegionProxy.java:159)
        at org.apache.geode.internal.cache.LocalRegion.serverPut(LocalRegion.java:3004)
        at org.apache.geode.internal.cache.DistributedRegion.cacheWriteBeforePut(DistributedRegion.java:2614)
        at org.apache.geode.internal.cache.AbstractRegionMap.invokeCacheWriter(AbstractRegionMap.java:3069)
        at org.apache.geode.internal.cache.AbstractRegionMap.basicPut(AbstractRegionMap.java:2828)
        at org.apache.geode.internal.cache.LocalRegion.virtualPut(LocalRegion.java:5622)
        at org.apache.geode.internal.cache.DistributedRegion.virtualPut(DistributedRegion.java:369)
        at org.apache.geode.internal.cache.LocalRegionDataView.putEntry(LocalRegionDataView.java:152)
        at org.apache.geode.internal.cache.LocalRegion.basicPut(LocalRegion.java:5051)
        at org.apache.geode.internal.cache.LocalRegion.validatedPut(LocalRegion.java:1583)
        at org.apache.geode.internal.cache.LocalRegion.put(LocalRegion.java:1570)
        at org.apache.geode.internal.cache.AbstractRegion.put(AbstractRegion.java:325)
        at org.apache.geode.internal.cache.tier.sockets.ClientServerMiscDUnitTest.putIntoRegion(ClientServerMiscDUnitTest.java:1445)
        at org.apache.geode.internal.cache.tier.sockets.ClientServerMiscDUnitTest.testOnServerMothods(ClientServerMiscDUnitTest.java:1423)
        at org.apache.geode.internal.cache.tier.sockets.ClientServerMiscDUnitTest.testOnSeverMethodsWithCachingProxyClientHA(ClientServerMiscDUnitTest.java:1398)

Looking at the code, it looks like it just has a low read timeout of 1 second in _createClientCache. So on a slow system this test could just timeout and throw this error. We should up the timeout.