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

Review request for geode, Bruce Schuchardt and Dan Smith.


Bugs: GEODE-774
    https://issues.apache.org/jira/browse/GEODE-774


Repository: geode


Description
-------

When a wan batch is retried, the operation will be rejected by primary bucket 
because it has seen it. But it will still distribute to the secondary bucket. 
Sometimes, the event could be accepted due to eventtracker timeout and clean up 
the recorded events. But this event does not contain a version tag and can 
never find its version tag in the system. 

The fix is not to distribute in this case. 

Following checking have been used in many other places:
if (event.getVersionTag() != null && event.getVersionTag().hasValidVersion())

I just introduced a few more.


Diffs
-----

  
gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/BucketRegion.java
 f487bd5 
  
gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/DistributedRegion.java
 92b585a 

Diff: https://reviews.apache.org/r/42099/diff/


Testing
-------

The in-house hydra test to reproduce it. 
precheckin.
A lot of wan hydra tests.


Thanks,

xiaojian zhou

Reply via email to