----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45881/#review127668 -----------------------------------------------------------
Ship it! Ship It! - Dan Smith On April 7, 2016, 5:34 p.m., anilkumar gingade wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45881/ > ----------------------------------------------------------- > > (Updated April 7, 2016, 5:34 p.m.) > > > Review request for geode, anilkumar gingade, Barry Oglesby, Jason Huynh, > nabarun nag, Dan Smith, and xiaojian zhou. > > > Repository: geode > > > Description > ------- > > GEODE-704: CI failure: > HAInterestPart2DUnitTest.testRefreshEntriesFromPrimaryWhenDSMDetectsServerLive > > In the test client re-registers the interests after cache-server is started > and expects entries to be refreshed (it stops and starts the cache-server, > without closing the cache)... > It looks like we are not waiting enough to see the client registration is > complete and events are refreshed (the re-registration happens with > redundancy satisfier thread). > > Added wait/check to see interest registration is complete before check is > done for the cache entries... > > > Diffs > ----- > > > geode-core/src/test/java/com/gemstone/gemfire/internal/cache/tier/sockets/HAInterestPart2DUnitTest.java > 02c3e71 > > Diff: https://reviews.apache.org/r/45881/diff/ > > > Testing > ------- > > Testing: > Run the test 100+ times. > > > Thanks, > > anilkumar gingade > >
