geode-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Bruce Schuchardt <>
Subject Review Request 59422: GEODE-2954 Old client gets null memberID in cache listener
Date Fri, 19 May 2017 22:48:25 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for geode, Galen O'Sullivan, Hitesh Khamesra, and Udo Kohlmeyer.

Bugs: GEODE-2954

Repository: geode


I've added a new test that demonstrates that a new-version server sends an EventID to a client
that the client is unable to deserialize completely.  It gets an error when deserializing
its member ID, causing cache listeners to get a null when requesting the ID of the member
that effected the change.

The fix is to reserialize the member ID in EventID.toData if the destination stream is for
an older version, such as a 1.1.0 client.  This ensures the proper on-wire format is used
for that version of Geode.

I've also bumped up the version ordinal for 1.2 since version 59 is marked as unusable in

I've changed the Banner to show the version ordinal because the other version information
in the banner isn't completely trustworthy.  It looks for a file
on the classpath to get this information and so it may not get it from the Geode jar file
as expected.


  geode-core/src/main/java/org/apache/geode/internal/ b6a89bfc530fa5f4766f61e124839479dff2b664

  geode-core/src/main/java/org/apache/geode/internal/ 1c131e8d08fc4b3f8004ffaca78fb6fac910ee2b

  geode-core/src/main/java/org/apache/geode/internal/cache/ 87835ffa5a9782fecec5f6ae7adfe9829ac2fc26




precheckin, new unit test.  I have to fix up the database for AnalyzeSerializablesJUnitTest
- I'm not including that in this review's diff.


Bruce Schuchardt

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message