----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58853/#review173406 -----------------------------------------------------------
geode-core/src/main/java/org/apache/geode/internal/cache/EventTracker.java Lines 321 (patched) <https://reviews.apache.org/r/58853/#comment246404> Do we need concurrency check here...We don't do that for non bulk op... geode-core/src/main/java/org/apache/geode/internal/cache/EventTracker.java Line 548 (original), 551 (patched) <https://reviews.apache.org/r/58853/#comment246409> Can this become null? geode-core/src/main/java/org/apache/geode/internal/cache/EventTracker.java Line 552 (original), 555 (patched) <https://reviews.apache.org/r/58853/#comment246408> If debug is enabled; we don't get into else part...Is this expected... - anilkumar gingade 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 > >