-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54003/
-----------------------------------------------------------
(Updated Nov. 29, 2016, 9:12 p.m.)
Review request for geode, Galen O'Sullivan, Hitesh Khamesra, and Udo Kohlmeyer.
Changes
-------
This updated diff removes some fields in InternalDistributedMember that were
duplicated in GMSMember, reducing the size of membership IDs. This also rolls
the product version forward so that we can retain backward compatibility with
the 1.0 release of Geode.
Bugs: geode-2137
https://issues.apache.org/jira/browse/geode-2137
Repository: geode
Description
-------
This removes the UUID and member-weight bytes from the serialized form of event
IDs, version tags and client IDs while retaining them for server IDs in general.
Diffs (updated)
-----
geode-core/src/main/java/org/apache/geode/distributed/internal/membership/InternalDistributedMember.java
0d4fd9e9a8d22266635a26c77d8dcb61a343a628
geode-core/src/main/java/org/apache/geode/distributed/internal/membership/MemberAttributes.java
75cdd498b5879e9ec91418447819b65737713449
geode-core/src/main/java/org/apache/geode/distributed/internal/membership/NetMember.java
4db207fb118a58c778b26dc6b104c8b9568d7112
geode-core/src/main/java/org/apache/geode/distributed/internal/membership/gms/GMSMember.java
670d62b54f12182420c60408ffff6813e3be34fa
geode-core/src/main/java/org/apache/geode/distributed/internal/membership/gms/GMSMemberFactory.java
c84690a441abec4eaab2d9808d1925b74728bfdd
geode-core/src/main/java/org/apache/geode/internal/Version.java
fabb3a74391e3c9aae415eb6f434ad942ee014fc
geode-core/src/main/java/org/apache/geode/internal/cache/persistence/DiskStoreID.java
e94b081d7559e6bea3dde2b9cbf6ebe568842193
geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/CommandInitializer.java
0fe44cf589f290d9c2504868264982264d038ac0
geode-core/src/main/java/org/apache/geode/internal/cache/versions/VersionSource.java
bfbda2f9b030bef40940d165c2711f913c5caf2b
geode-core/src/main/java/org/apache/geode/internal/cache/versions/VersionTag.java
fa2c15669e97cddbaedd4ac326d47a01dd288434
geode-core/src/test/java/org/apache/geode/distributed/LocatorDUnitTest.java
128f970ff3d93dbd0010dce88e9fd123239b0a80
geode-core/src/test/java/org/apache/geode/distributed/LocatorJUnitTest.java
d1e56c46b78a28d26278c074fa52c49a4992e954
geode-core/src/test/java/org/apache/geode/distributed/RoleDUnitTest.java
d8047038c8977dbaa7a99bed9fe8cce9a1f241b4
geode-core/src/test/java/org/apache/geode/distributed/internal/DistributionManagerDUnitTest.java
b38a769b6804d2da8f230d33ae15b0a36c53b422
geode-core/src/test/java/org/apache/geode/distributed/internal/InternalDistributedSystemJUnitTest.java
49e8fa6be557b0f7aa0e7ca91733d2a8fd5ce00d
geode-core/src/test/java/org/apache/geode/distributed/internal/membership/gms/fd/GMSHealthMonitorJUnitTest.java
ebf87be77e84fe3f97fdf5f51305942986f00152
geode-core/src/test/java/org/apache/geode/test/dunit/standalone/ProcessManager.java
c3c33d04014b173d45bf4bbf7c74a10fc0cc1335
Diff: https://reviews.apache.org/r/54003/diff/
Testing
-------
precheckin, membership and client/server integration testing
Thanks,
Bruce Schuchardt
