You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@geode.apache.org by Darrel Schneider <ds...@pivotal.io> on 2015/08/06 01:52:02 UTC

Review Request 37148: fix race in testEntryTtlDestroyEvent

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

Review request for geode and Kirk Lund.


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


Repository: geode


Description
-------

A race existed that caused no-ack regions to fail because they
did not see the entry expire.
This race was fixed by suspending expiration until the create
is seen on the remote side and then after permitting expiration
using a wait for the expire to happen.


Diffs
-----

  gemfire-core/src/test/java/com/gemstone/gemfire/cache30/MultiVMRegionTestCase.java 5e605dd11cd010053fb988dcc5d05a403cf288f2 

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


Testing
-------

precheckin


Thanks,

Darrel Schneider

Re: Review Request 37148: fix race in testEntryTtlDestroyEvent

Posted by Kirk Lund <ki...@gmail.com>.

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

Ship it!


Ship It!

- Kirk Lund


On Aug. 5, 2015, 11:52 p.m., Darrel Schneider wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37148/
> -----------------------------------------------------------
> 
> (Updated Aug. 5, 2015, 11:52 p.m.)
> 
> 
> Review request for geode and Kirk Lund.
> 
> 
> Bugs: GEODE-179
>     https://issues.apache.org/jira/browse/GEODE-179
> 
> 
> Repository: geode
> 
> 
> Description
> -------
> 
> A race existed that caused no-ack regions to fail because they
> did not see the entry expire.
> This race was fixed by suspending expiration until the create
> is seen on the remote side and then after permitting expiration
> using a wait for the expire to happen.
> 
> 
> Diffs
> -----
> 
>   gemfire-core/src/test/java/com/gemstone/gemfire/cache30/MultiVMRegionTestCase.java 5e605dd11cd010053fb988dcc5d05a403cf288f2 
> 
> Diff: https://reviews.apache.org/r/37148/diff/
> 
> 
> Testing
> -------
> 
> precheckin
> 
> 
> Thanks,
> 
> Darrel Schneider
> 
>