----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53094/#review153607 -----------------------------------------------------------
geode-core/src/main/java/org/apache/geode/internal/cache/AbstractRegionMap.java (line 1516) <https://reviews.apache.org/r/53094/#comment223014> I don't understand this comment. There doesn't seem to be a check for concurrency-checks being enabled & you've removed the check for a tombstone. - Bruce Schuchardt On Oct. 21, 2016, 6:41 p.m., Hitesh Khamesra wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53094/ > ----------------------------------------------------------- > > (Updated Oct. 21, 2016, 6:41 p.m.) > > > Review request for geode, Bruce Schuchardt, Darrel Schneider, and Udo > Kohlmeyer. > > > Repository: geode > > > Description > ------- > > ExpirationManager tracks the regionEntry as reference to expiryTask. It > assumes, it is unique for that key. But consistency check mechanism keeps > that regionEntry around and that enforces re-use of that for new update. That > introduces the race condition between expiry thread and user thread, it endup > not scheduling the entry for expiration. As a fix, now "consistency check > mechanism" unschedules the expiry task to avoid this race condition. > > > Diffs > ----- > > > geode-core/src/main/java/org/apache/geode/internal/cache/AbstractRegionMap.java > e02c7e1 > > geode-core/src/main/java/org/apache/geode/internal/cache/EntryEventImpl.java > d059aab > > geode-core/src/main/java/org/apache/geode/internal/cache/EntryExpiryTask.java > 0c20d32 > geode-core/src/main/java/org/apache/geode/internal/cache/LocalRegion.java > ac4c705 > > Diff: https://reviews.apache.org/r/53094/diff/ > > > Testing > ------- > > > Thanks, > > Hitesh Khamesra > >