geode-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Bruce Schuchardt <>
Subject Re: Review Request 60446: Events do not get removed from the client queue for 1.0 clients
Date Tue, 27 Jun 2017 19:11:14 GMT

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

(Updated June 27, 2017, 12:11 p.m.)

Review request for geode and Barry Oglesby.


Addressing some of the recent code hygene comments

Bugs: GEODE-3072

Repository: geode


EventID and ThreadIdentifier hold the serialized form of portions of an InternalDistributedMember
identifier.  This serialized form changed in v1.0.0 and was causing .equals and .hashCode
for these objects to return false when they should have returned true owing to additional
data being in the serialized form of the identifier.

This change set modifies the equals and hashCode methods of the classes to ensure that they
return the correct results in the presence of this additional ID data.

I created a dunit test to reproduce the problem but I think the behavior of HARegionQueues
isn't reliable enough to check in that test.  I'm afraid that it would end up being a "flaky"
test that fails periodically.  Instead, I'm relying on a new junit test and the test that
Barry already checked in.

Note: I've also included two other things in this change set.

First, LocalRegion.dumpBackingMap() is a test-hook that should log its results at "info" level.
 I used it in debugging this problem.

Second, I've added a new backward-compatibility version so now we have 100 and 110.

Diffs (updated)

  geode-core/src/main/java/org/apache/geode/internal/cache/ a1b4a9d5684d0148bd9e72c00c674afa299b2b9d

  geode-core/src/main/java/org/apache/geode/internal/cache/ 4d2ddc11bb1bd36446dc30b2033a6bc1ed455c98

  geode-core/src/main/java/org/apache/geode/internal/cache/ha/ ec165a5af210966241fdc1cee81702231557cc8c

  geode-old-versions/build.gradle d85eb0b7dea6d3fa6b19a0030e64433cb4cb7cb9 





Bruce Schuchardt

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