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

Reply via email to