Browse Source

Use the default restart intensity in all supervisors

This reduces from 10 restarts in 10 seconds to 1 restart
in 5 seconds. This is the new default in OTP 18, and it
fits the kinds of processes that Ranch deals with:

* Supervisors: default makes sense.

* Acceptors: they crash on socket error. They'll probably
  crash again if the socket didn't change.

* Connection processes: they are never restarted.
Loïc Hoguin 9 years ago
parent
commit
ae8ff1fc0f
3 changed files with 3 additions and 3 deletions
  1. 1 1
      src/ranch_acceptors_sup.erl
  2. 1 1
      src/ranch_listener_sup.erl
  3. 1 1
      src/ranch_sup.erl

+ 1 - 1
src/ranch_acceptors_sup.erl

@@ -46,7 +46,7 @@ init([Ref, NbAcceptors, Transport, TransOpts]) ->
 			LSocket, Transport, ConnsSup
 		]}, permanent, brutal_kill, worker, []}
 			|| N <- lists:seq(1, NbAcceptors)],
-	{ok, {{one_for_one, 10, 10}, Procs}}.
+	{ok, {{one_for_one, 1, 5}, Procs}}.
 
 -spec listen_error(any(), module(), any(), atom()) -> no_return().
 listen_error(Ref, Transport, TransOpts2, Reason) ->

+ 1 - 1
src/ranch_listener_sup.erl

@@ -39,4 +39,4 @@ init({Ref, NbAcceptors, Transport, TransOpts, Protocol}) ->
 				[Ref, NbAcceptors, Transport, TransOpts]},
 			permanent, infinity, supervisor, [ranch_acceptors_sup]}
 	],
-	{ok, {{rest_for_one, 10, 10}, ChildSpecs}}.
+	{ok, {{rest_for_one, 1, 5}, ChildSpecs}}.

+ 1 - 1
src/ranch_sup.erl

@@ -29,4 +29,4 @@ init([]) ->
 		{ranch_server, {ranch_server, start_link, []},
 			permanent, 5000, worker, [ranch_server]}
 	],
-	{ok, {{one_for_one, 10, 10}, Procs}}.
+	{ok, {{one_for_one, 1, 5}, Procs}}.