-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58813/
-----------------------------------------------------------

(Updated May 5, 2017, 3:57 a.m.)


Review request for geode, Darrel Schneider, Eric Shu, and Lynn Gallinat.


Changes
-------

Fixing unit test failure ProxyJUnitTest.testExpiration()


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 
0c967c9 
  geode-core/src/main/java/org/apache/geode/internal/cache/LocalRegion.java 
2dec53b 
  
geode-core/src/test/java/org/apache/geode/internal/cache/DistributedRegionSearchLoadJUnitTest.java
 PRE-CREATION 


Diff: https://reviews.apache.org/r/58813/diff/5/

Changes: https://reviews.apache.org/r/58813/diff/4-5/


Testing
-------

Manual testing.
Running new unit test (added) with and without changes.
precheckin in progress.


Thanks,

anilkumar gingade

Reply via email to