----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40085/ -----------------------------------------------------------
(Updated Nov. 9, 2015, 4:41 p.m.) Review request for geode, anilkumar gingade, Hitesh Khamesra, Jason Huynh, Jianxia Chen, and Lynn Gallinat. Changes ------- adding unit test for GMSJoinLeave change Repository: geode Description ------- In integration testing members were, under heavy load, being forced out of the distributed system in spite of the new tcp/ip final-checks. This change set adds a recent-activity check to the doTCPCheckMember method of GMSHealthMonitor. It also corrects this method to use SocketCreator with a background timeout task in case the tcp/ip socket timeout fails to work (as it sometimes does). During testing I also saw that GMSJoinLeave was not initiating a forced-disconnect if it received a membership view showing that it was no longer in the distributed system and modified it to do so. Diffs (updated) ----- gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/fd/GMSHealthMonitor.java 0307f5badb27148a772c32c506688c6647965bc8 gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/membership/GMSJoinLeave.java 4b1993f5be3de4b55d344a1285108ea716fbbfb9 gemfire-core/src/test/java/com/gemstone/gemfire/distributed/internal/membership/gms/membership/GMSJoinLeaveJUnitTest.java 89282c05a09792083b0f3cab7a7da882641d7012 Diff: https://reviews.apache.org/r/40085/diff/ Testing ------- unit tests, integration testing Thanks, Bruce Schuchardt
