----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54184/#review157355 -----------------------------------------------------------
Ship it! Ship It! - Dan Smith On Nov. 29, 2016, 10:43 p.m., Jason Huynh wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54184/ > ----------------------------------------------------------- > > (Updated Nov. 29, 2016, 10:43 p.m.) > > > Review request for geode, Barry Oglesby and Dan Smith. > > > Repository: geode > > > Description > ------- > > Also fixed a possibly flaky test by removing the sleeps and having it check > the actual size of the queue (including secondary buckets) > Removed some unnecessary sleep calls > > > Diffs > ----- > > > geode-core/src/main/java/org/apache/geode/internal/cache/PartitionedRegionDataStore.java > 2e9fdbc > > geode-core/src/main/java/org/apache/geode/internal/cache/wan/parallel/ConcurrentParallelGatewaySenderQueue.java > 603fd6c > > geode-core/src/main/java/org/apache/geode/internal/cache/wan/parallel/ParallelGatewaySenderQueue.java > 553847d > > geode-wan/src/test/java/org/apache/geode/internal/cache/wan/WANTestBase.java > f9c18ec > > geode-wan/src/test/java/org/apache/geode/internal/cache/wan/concurrent/ConcurrentParallelGatewaySenderOperation_2_DUnitTest.java > 931a49c > > geode-wan/src/test/java/org/apache/geode/internal/cache/wan/parallel/ParallelWANPropagationConcurrentOpsDUnitTest.java > 0ce6782 > > Diff: https://reviews.apache.org/r/54184/diff/ > > > Testing > ------- > > > Thanks, > > Jason Huynh > >
