> On Nov. 22, 2016, 11:33 p.m., Hitesh Khamesra wrote: > > Does that cover the case when event(or put) happened through peer(or server)
It does cover peer-to-peer. One thing that I need to do though is bump the current Version and make sure this works with Geode 1.0.0. - Bruce ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54003/#review156662 ----------------------------------------------------------- On Nov. 22, 2016, 11:21 p.m., Bruce Schuchardt wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54003/ > ----------------------------------------------------------- > > (Updated Nov. 22, 2016, 11:21 p.m.) > > > Review request for geode, Galen O'Sullivan, Hitesh Khamesra, and Udo > Kohlmeyer. > > > 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 > ----- > > > 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/NetMember.java > 4db207fb118a58c778b26dc6b104c8b9568d7112 > > geode-core/src/main/java/org/apache/geode/distributed/internal/membership/gms/GMSMember.java > 670d62b54f12182420c60408ffff6813e3be34fa > geode-core/src/test/java/org/apache/geode/distributed/LocatorDUnitTest.java > 128f970ff3d93dbd0010dce88e9fd123239b0a80 > > geode-core/src/test/resources/org/apache/geode/codeAnalysis/sanctionedDataSerializables.txt > 23162e5d42d24e6febaed8e3cbb14c4b7256926f > > Diff: https://reviews.apache.org/r/54003/diff/ > > > Testing > ------- > > precheckin, membership and client/server integration testing > > > Thanks, > > Bruce Schuchardt > >
