Skip to content

Commit

Permalink
Moved mongoose_node_num:set_node_num away from try..catch
Browse files Browse the repository at this point in the history
  • Loading branch information
arcusfelis committed Jun 29, 2023
1 parent a9ab441 commit 80077e2
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions src/mongoose_cets_discovery_rdbms.erl
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,11 @@ init(Opts = #{cluster_name := _, node_name_to_insert := _}) ->
-spec get_nodes(state()) -> {cets_discovery:get_nodes_result(), state()}.
get_nodes(State = #{cluster_name := ClusterName, node_name_to_insert := Node}) ->
try
{Num, Nodes, Info} = try_register(ClusterName, Node),
mongoose_node_num:set_node_num(Num),
{{ok, Nodes}, State#{last_query_info => Info}}
try_register(ClusterName, Node)
of
{Num, Nodes, Info} ->
mongoose_node_num:set_node_num(Num),
{{ok, Nodes}, State#{last_query_info => Info}}
catch Class:Reason:Stacktrace ->
?LOG_ERROR(#{what => discovery_failed_select, class => Class,
reason => Reason, stacktrace => Stacktrace}),
Expand Down

0 comments on commit 80077e2

Please sign in to comment.