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

(Updated July 25, 2017, 7:03 p.m.)


Review request for geode, anilkumar gingade, Darrel Schneider, and Dan Smith.


Changes
-------

This includes changes in response to review comments and a dunit test.


Repository: geode


Description
-------

An online backup was not taking a snapshot of a single point in time. The 
solution is for operations that change the disk files to acquire the backup 
lock, causing them to wait until backup has rolled the op logs.


Diffs (updated)
-----

  geode-core/src/main/java/org/apache/geode/internal/cache/BackupLock.java 
4b4fb10 
  geode-core/src/main/java/org/apache/geode/internal/cache/DiskInitFile.java 
0925d28 
  geode-core/src/main/java/org/apache/geode/internal/cache/DiskStoreImpl.java 
3e97d0e 
  geode-core/src/main/java/org/apache/geode/internal/cache/Oplog.java 5399d5a 
  
geode-core/src/test/java/org/apache/geode/internal/cache/persistence/BackupPrepareAndFinishMsgDUnitTest.java
 PRE-CREATION 
  
geode-core/src/test/resources/org/apache/geode/codeAnalysis/excludedClasses.txt 
db234e0 
  
geode-core/src/test/resources/org/apache/geode/codeAnalysis/sanctionedSerializables.txt
 e5a111c 


Diff: https://reviews.apache.org/r/60875/diff/2/

Changes: https://reviews.apache.org/r/60875/diff/1-2/


Testing
-------

Precheckin.


Thanks,

Lynn Gallinat

Reply via email to