nickva closed pull request #1627: Switch rexi server_per_node to true by default URL: https://github.com/apache/couchdb/pull/1627
This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/rel/overlay/etc/default.ini b/rel/overlay/etc/default.ini index dc2e51cc0d..ba2a498ebc 100644 --- a/rel/overlay/etc/default.ini +++ b/rel/overlay/etc/default.ini @@ -187,7 +187,7 @@ port = 6984 ; [rexi] ; buffer_count = 2000 -; server_per_node = false +; server_per_node = true ; [global_changes] ; max_event_delay = 25 diff --git a/src/rexi/src/rexi_server_mon.erl b/src/rexi/src/rexi_server_mon.erl index 86fecaff67..cfe1144cea 100644 --- a/src/rexi/src/rexi_server_mon.erl +++ b/src/rexi/src/rexi_server_mon.erl @@ -68,6 +68,8 @@ cluster_stable(Server) -> init(ChildMod) -> {ok, _Mem3Cluster} = mem3_cluster:start_link(?MODULE, self(), ?CLUSTER_STABILITY_PERIOD_SEC, ?CLUSTER_STABILITY_PERIOD_SEC), + start_servers(ChildMod), + couch_log:notice("~s : started servers", [ChildMod]), {ok, ChildMod}. diff --git a/src/rexi/src/rexi_utils.erl b/src/rexi/src/rexi_utils.erl index 11dbb252ae..960318418a 100644 --- a/src/rexi/src/rexi_utils.erl +++ b/src/rexi/src/rexi_utils.erl @@ -16,8 +16,8 @@ %% @doc Return a rexi_server id for the given node. server_id(Node) -> - case config:get("rexi", "server_per_node", "false") of - "true" -> + case config:get_boolean("rexi", "server_per_node", true) of + true -> list_to_atom("rexi_server_" ++ atom_to_list(Node)); _ -> rexi_server ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services