----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52789/#review152352 -----------------------------------------------------------
Ship it! Ship It! - Dan Smith On Oct. 12, 2016, 5:21 p.m., nabarun nag wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52789/ > ----------------------------------------------------------- > > (Updated Oct. 12, 2016, 5:21 p.m.) > > > 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 > >