----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47358/ -----------------------------------------------------------
Review request for geode, Hitesh Khamesra, Jon Chen, Udo Kohlmeyer, and Dan Smith. Repository: geode Description ------- When a locator starts up it has no knowledge of what cache servers are in the system. It doesn't know this until it has received at least on load update from the servers. This can cause clients to throw a NoAvailableServersException, and it's probably why we see these inexplicable exceptions in test failures from time to time. I've rewritten ServerLocator.processRequest() to allow early handling of locator location requests and to disallow early handling of server location requests. Diffs ----- geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/LocatorStats.java d42a2b45a4383931318e97bf0575e084ef4aa7ed geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/ServerLocator.java e535b97688e74ce30b259cbe78c9921db9ba9c13 geode-core/src/test/java/com/gemstone/gemfire/distributed/internal/ServerLocatorJUnitTest.java bb83df95d20b7146509a00a10a3b12cc923c97c4 Diff: https://reviews.apache.org/r/47358/diff/ Testing ------- New unit test. Precheckin is underway. Thanks, Bruce Schuchardt
