Skip to content

Commit

Permalink
Return empty list instead of undefined when extracting membership / w…
Browse files Browse the repository at this point in the history
…itness membership from an existing config

Summary: as title

Differential Revision:
D56559761

Privacy Context Container: L1141030

fbshipit-source-id: d40f44fd71e5d97235656479e0ff55fe84085ded
  • Loading branch information
Jose-Angel Herrero Bajo authored and facebook-github-bot committed Apr 25, 2024
1 parent 65f004e commit 096a068
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions src/wa_raft_server.erl
Original file line number Diff line number Diff line change
Expand Up @@ -201,17 +201,17 @@ start_link(#raft_options{server_name = Name} = Options) ->
%% RAFT Server - Cluster Config API
%% ==================================================

-spec get_config_members(Config :: config()) -> undefined | [#raft_identity{}].
-spec get_config_members(Config :: config()) -> [#raft_identity{}].
get_config_members(#{version := ?RAFT_CONFIG_CURRENT_VERSION, membership := Members}) ->
[#raft_identity{name = Name, node = Node} || {Name, Node} <- Members];
get_config_members(_Config) ->
undefined.
[].

-spec get_config_witnesses(Config :: config()) -> undefined | [#raft_identity{}].
-spec get_config_witnesses(Config :: config()) -> [#raft_identity{}].
get_config_witnesses(#{version := ?RAFT_CONFIG_CURRENT_VERSION, witness := Witnesses}) ->
[#raft_identity{name = Name, node = Node} || {Name, Node} <- Witnesses];
get_config_witnesses(_Config) ->
undefined.
[].

-spec make_config(Members :: [#raft_identity{}]) -> config().
make_config(Members) ->
Expand Down

0 comments on commit 096a068

Please sign in to comment.