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

Review request for geode, Barry Oglesby, Jason Huynh, Dan Smith, and xiaojian 
zhou.


Repository: geode


Description
-------

Scenario without the fix:

1. The test creates receivers and senders.
2. It initiates put operation on the sender as an async operation.
3. Immediately it destroys the receivers regions.
4. It assumes that sender must not have completed the transmission because 
receiver region is destroyed.
5. It checks that the sender's queue is not empty 


Problem: Its assumption that senders have not completed the transmission before 
the receiver's region is destroyed is wrong. It may have completed the 
transmission and the test fails because sender queues are  empty.

Solution: We slow down the receivers using addListenerToSleepAfterCreateEvent, 
hence we make sure that the senders are not able to complete the transmission 
before the receiver regions are destroyed. This function was used to solve 
similar timing related bugs in WAN.


Diffs
-----

  
geode-wan/src/test/java/org/apache/geode/internal/cache/wan/concurrent/ConcurrentWANPropagation_1_DUnitTest.java
 8bfd8e7 

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


Testing
-------

* Precheckin
* 100 runs on IntelliJ


Thanks,

nabarun nag

Reply via email to