Skip to content

Commit ae171b5

Browse files
SimonUngemergify[bot]
authored andcommitted
Dont handle the exception just let it out there
(cherry picked from commit cdeabe2)
1 parent 73742e4 commit ae171b5

File tree

1 file changed

+2
-12
lines changed

1 file changed

+2
-12
lines changed

deps/rabbit/src/rabbit_db_cluster.erl

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -227,18 +227,8 @@ join(RemoteNode, NodeType)
227227
"with node ~tp, but ~tp disagrees. ~tp will ask "
228228
"to leave the cluster and try again.",
229229
[RemoteNode, node(), node(), node()]),
230-
try
231-
ok = rabbit_mnesia:leave_discover_cluster(RemoteNode),
232-
join(RemoteNode, NodeType)
233-
catch
234-
Exception ->
235-
%% Should we handle the catched error - my reasoning for
236-
%% ignoring it is that the error we want to show is the
237-
%% issue of joinging the cluster, not the potential error
238-
%% of leaving the cluster.
239-
rabbit_log:error(Msg),
240-
Exception
241-
end
230+
ok = rabbit_mnesia:leave_discover_cluster(RemoteNode),
231+
join(RemoteNode, NodeType)
242232
end
243233
end.
244234

0 commit comments

Comments
 (0)