[GitHub] [geode] kirklund commented on issue #3904: GEODE-7070: Remove AlertAppender usage from GMSMembershipManager

2019-08-16 Thread GitHub
Closing this PR to investigate potential for appender recursion more. [ Full content available at: https://github.com/apache/geode/pull/3904 ] This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode] kirklund commented on issue #3904: GEODE-7070: Remove AlertAppender usage from GMSMembershipManager

2019-08-15 Thread GitHub
@bschuchardt (Not sure if my editing made this more readable or not) I'm actually a little worried about merging this change in. Without this change, LynnG encountered a failure which I think might occur more if we merge this in. LynnG's analysis: In this run we are expecting

[GitHub] [geode] kirklund commented on issue #3904: GEODE-7070: Remove AlertAppender usage from GMSMembershipManager

2019-08-15 Thread GitHub
@bschuchardt I'm actually a little worried about merging this change in. Without this change, LynnG encountered a failure which I think might occur more if we merge this in. In this run we are expecting ForcedDisconnects, but they don't occur until after the hang is declared. The difference

[GitHub] [geode] kirklund commented on issue #3904: GEODE-7070: Remove AlertAppender usage from GMSMembershipManager

2019-08-15 Thread GitHub
@bschuchardt I'm actually a little worried about merging this change in. Without this change, LynnG encountered a failure which I think might occur more if we merge this in. In this run we are expecting ForcedDisconnects, but they don't occur until after the hang is declared. The difference