----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46056/#review128521 -----------------------------------------------------------
Ship it! Ship It! - Udo Kohlmeyer On April 12, 2016, 3:51 p.m., Bruce Schuchardt wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46056/ > ----------------------------------------------------------- > > (Updated April 12, 2016, 3:51 p.m.) > > > Review request for geode, Hitesh Khamesra, Jianxia Chen, and Udo Kohlmeyer. > > > Bugs: GEODE-1178 > https://issues.apache.org/jira/browse/GEODE-1178 > > > Repository: geode > > > Description > ------- > > This has been reported to JGroups. While they're deciding what to do about > it I have coded a workaround in our StatRecorder class. StatRecorder sits in > the JGroups stack just above the transport protocol that is throwing this > exception from its down() method. StatRecorder will now catch the exception > and, after sleeping a short amount of time (10ms) it will retry as long as > the Manager is not shutting down. > > > Diffs > ----- > > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/Services.java > 4484c00adfbaa0d43e223515699201b4a2c6f102 > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/fd/GMSHealthMonitor.java > 510c5a8609d4d178da7b65667f6e244cbd1c01c9 > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/messenger/JGroupsMessenger.java > 2dfeeaaabcac9af4bd31f8359e1e62cdcdee41cc > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/messenger/StatRecorder.java > e29d71eb1c877153bfe99cef27566b76d6bea80f > > geode-core/src/test/java/com/gemstone/gemfire/distributed/internal/membership/gms/membership/StatRecorderJUnitTest.java > 91d4a5738fcbf9a073e894afe4d0382e077a2a61 > > Diff: https://reviews.apache.org/r/46056/diff/ > > > Testing > ------- > > New unit test. Precheckin is underway. > > > Thanks, > > Bruce Schuchardt > >