----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59242/ -----------------------------------------------------------
Review request for geode, Galen O'Sullivan, Hitesh Khamesra, and Udo Kohlmeyer. Bugs: GEODE-2915 https://issues.apache.org/jira/browse/GEODE-2915 Repository: geode Description ------- The fix for GEODE_2875 has exacerbated this problem, which we used to only see in cases where disable-tcp=true or when multicast was enabled. The problem is that JGroupsMessenger is not sending the "vmkind" of the sender in message headers. This part of the header comes from GMSMember.writeEssentialData(). I've changed it here to include the vmKind if the recipient isn't using geode 1.0, which doesn't expect the version byte. Diffs ----- geode-core/src/main/java/org/apache/geode/distributed/internal/membership/InternalDistributedMember.java 41c85d6421c8283163b70f2a560c8e4cbb02f2cc geode-core/src/main/java/org/apache/geode/distributed/internal/membership/gms/GMSMember.java b7079f8bc20a0e58949b69b9f0174a26af1a9b86 geode-core/src/main/java/org/apache/geode/distributed/internal/membership/gms/mgr/GMSMembershipManager.java 0476bbbfa4a1480d3b31a052e98dc62d9f0e3867 geode-core/src/main/java/org/apache/geode/internal/i18n/LocalizedStrings.java 85042da54f5a2a772d39ba450110073e14a30196 geode-core/src/test/java/org/apache/geode/distributed/internal/membership/gms/GMSMemberJUnitTest.java f471ad99b56615a1935ccf52127960f4af763d7d Diff: https://reviews.apache.org/r/59242/diff/1/ Testing ------- new unit test. Precheckin is underway. I expect AnalyzeSerializables to fail & will need to update its sanctionedDataSerializables.txt record for GMSMember. Thanks, Bruce Schuchardt