don't erase aliases quite so eagerly on DOWN

since that can result in completely forgetting about a member when
there are still acks making their way around the right.

1 files changed, 9 insertions, 8 deletions

diff --git a/src/gm.erl b/src/gm.erl
index 5e9ed460fa..acc9444706 100644
--- a/src/gm.erl
+++ b/src/gm.erl
@@ -709,15 +709,16 @@ handle_info({'DOWN', MRef, process, _Pid, Reason},
         _ ->
             View1 = group_to_view(record_dead_member_in_group(
                                     Member, GroupName, TxnFun)),
-            {Result, State1} = maybe_erase_aliases(State, View1),
             handle_callback_result(
-              {Result,
-               case alive_view_members(View1) of
-                   [Self] -> State1 #state {
-                               members_state = blank_member_state(),
-                               confirms      = purge_confirms(Confirms) };
-                   _      -> State1
-               end})
+              case alive_view_members(View1) of
+                  [Self] ->
+                      {Result, State1} = maybe_erase_aliases(State, View1),
+                      {Result, State1 #state {
+                                 members_state = blank_member_state(),
+                                 confirms      = purge_confirms(Confirms) }};
+                  _ ->
+                      change_view(View1, State)
+              end)
     end.