----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52896/ -----------------------------------------------------------
Review request for geode, Bruce Schuchardt, Darrel Schneider, and Udo Kohlmeyer. Repository: geode Description ------- There was possibility that expiry thread destroying the entry and another thread doing update on same key. In this case expiry thread cancel's existing task and update thread adds expiry task. But this tasks are refer by regionEntry, which is same for both the threads. So in this case if expiry thread cancel's task after update thread then that entry will never expire. Diffs ----- geode-core/src/main/java/org/apache/geode/internal/cache/AbstractRegionMap.java 5861e9a geode-core/src/main/java/org/apache/geode/internal/cache/EntryEventImpl.java 6a964c0 geode-core/src/main/java/org/apache/geode/internal/cache/EntryExpiryTask.java 816f32f geode-core/src/main/java/org/apache/geode/internal/cache/LocalRegion.java a6951de Diff: https://reviews.apache.org/r/52896/diff/ Testing ------- Thanks, Hitesh Khamesra