----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47189/ -----------------------------------------------------------
(Updated May 11, 2016, 4:50 p.m.) Review request for geode, Hitesh Khamesra, Jianxia Chen, and Udo Kohlmeyer. Changes ------- Diff updated - no need to save the JoinResponse if the new member has already finished joining the distributed system Repository: geode Description ------- This reinstates the sending of JoinResponseMessages so that the new member can get the jgroups multicast digest. The JoinResponseMessages are sent after installing the new membership view, so JGroupsMessenger has been changed to use MERGE_VIEW instead of SET_VIEW to install the digest since it may have already received multicast messages from some members. Diffs (updated) ----- geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/membership/GMSJoinLeave.java 88e4d496e5d89f2b84d5e755fc6471c8790ed98f geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/messenger/JGroupsMessenger.java 4a54e8433ccf0bcfdacc3ccc6e790381da6e236a geode-core/src/test/java/com/gemstone/gemfire/distributed/internal/membership/gms/membership/GMSJoinLeaveJUnitTest.java 50bed13419d1ed68a405a9f5ed5d5543c3d98813 Diff: https://reviews.apache.org/r/47189/diff/ Testing ------- precheckin is underway. There is already a unit test checking that a digest is added to a JoinResponseMessage so I didn't need to write one. Thanks, Bruce Schuchardt
