----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39605/ -----------------------------------------------------------
(Updated Oct. 27, 2015, 9:25 p.m.) Review request for geode, anilkumar gingade, Bruce Schuchardt, Jianxia Chen, and Lynn Gallinat. Changes ------- Additional change to not clear from removedMember and leftMembers collections when a view is installed. Instead we will only remove from those collections if the new view does not contain the member ids. This should help with booting out members in a more timely fashion. Repository: geode Description ------- It looks like the coordinator node has left, sent a leave message but the member that is supposed to become coordinator next, ignores this leave message (as he is not yet the coordinator). The fix is to not ignore these leave messages and instead have each member track them, similar to removedMembers list. When the leave message is processed by non coordinators, the results will be the same. However if the non coordinator will now become coordinator, they can now process these leave messages Diffs (updated) ----- gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/NetView.java cb61a1b gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/fd/GMSHealthMonitor.java 774ab37 gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/membership/GMSJoinLeave.java 6d39a6a gemfire-core/src/test/java/com/gemstone/gemfire/distributed/internal/membership/gms/membership/GMSJoinLeaveJUnitTest.java 41b0df7 Diff: https://reviews.apache.org/r/39605/diff/ Testing ------- Ran precheckin but had a handful of failures due to suspect strings, will rerun Thanks, Jason Huynh
