----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58853/#review173386 -----------------------------------------------------------
Ship it! Ship It! - xiaojian zhou On April 28, 2017, 8:17 p.m., Eric Shu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58853/ > ----------------------------------------------------------- > > (Updated April 28, 2017, 8:17 p.m.) > > > Review request for geode, anilkumar gingade, Darrel Schneider, and Lynn > Gallinat. > > > Bugs: GEODE-2847 > https://issues.apache.org/jira/browse/GEODE-2847 > > > Repository: geode > > > Description > ------- > > Get correct version tags from recordedBulkOpVersionTags in eventTracker. > Do not remove the recordedBulkOpVersionTags prematurely. > Add the unit test which would fail without the fixes. > > > Diffs > ----- > > geode-core/src/main/java/org/apache/geode/internal/cache/EventTracker.java > 2ddfdc4 > geode-core/src/main/java/org/apache/geode/internal/cache/LocalRegion.java > 8c061b0 > > geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/PutAllPRMessage.java > 27f5aa0 > > geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/RemoveAllPRMessage.java > f4f6299 > > geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/ClientProxyMembershipID.java > 2cbf63b > > geode-core/src/test/java/org/apache/geode/internal/cache/EventTrackerTest.java > PRE-CREATION > > > Diff: https://reviews.apache.org/r/58853/diff/1/ > > > Testing > ------- > > precheckin. > > > Thanks, > > Eric Shu > >