----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54924/ -----------------------------------------------------------
(Updated Dec. 21, 2016, 11:40 p.m.) Review request for geode, Bruce Schuchardt, Jinmei Liao, Kevin Duling, and Kirk Lund. Changes ------- I found another place that needs to be fixed in the StartupMessage processing. The reply processor stopped waiting as soon as it received one reply, which means there was still race where we could continue member startup before receiving information about locators that have cluster configuration. With this latest fix, we will wait for all members to reply. Repository: geode Description ------- Call super.process last in StartupResponseWithVersionMessage, so that that threads waiting on the response will have the results of dm.addHostedLocators. Diffs (updated) ----- geode-core/src/main/java/org/apache/geode/distributed/internal/StartupMessageReplyProcessor.java de6e71fee7281df8997a76b1b8ce7b76ca72bfbf geode-core/src/main/java/org/apache/geode/distributed/internal/StartupResponseWithVersionMessage.java 6276d63a4dc496cc809bcb17c52fe4f80b362379 Diff: https://reviews.apache.org/r/54924/diff/ Testing ------- Thanks, Dan Smith
