----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58813/ -----------------------------------------------------------
(Updated May 3, 2017, 12:52 a.m.) Review request for geode, Darrel Schneider, Eric Shu, and Lynn Gallinat. Changes ------- Changes are made to get versionTag and the value from "re" atomically. Also refactored the findObjectInSystem(), moved relavent code to separate methods. Repository: geode Description ------- When client does a get() which results in adding an entry by calling loader on server side, the client event returned back is not updated with the version tag that is created with the new entry on server. This results in client having a different version tag than the server side entry. If client has registered event, and is concurrently updating the entry (from get() call and an register-event from server), it could result in data consistency between client and server. Diffs (updated) ----- geode-core/src/main/java/org/apache/geode/internal/cache/DistributedRegion.java 8cdc517 geode-core/src/test/java/org/apache/geode/internal/cache/DistributedRegionSearchLoadJUnitTest.java PRE-CREATION Diff: https://reviews.apache.org/r/58813/diff/3/ Changes: https://reviews.apache.org/r/58813/diff/2-3/ Testing ------- Manual testing. Running new unit test (added) with and without changes. precheckin in progress. Thanks, anilkumar gingade
