Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
MB-100: wait long enough for child processes shutdown
We spotted that we waited for memcached only for 1E6 milliseconds
which is less than we thought. We found it was caused by
supervisor_cushion timeout which makes sense to bump to infinity.

Change-Id: Ifc4751ab0b4e44648918a5c291be6299d3cf95db
Reviewed-on: http://review.couchbase.org/26651
Tested-by: Aliaksey Kandratsenka <alkondratenko@gmail.com>
Reviewed-by: Aliaksey Artamonau <aliaksiej.artamonau@gmail.com>
  • Loading branch information
Aliaksey Kandratsenka authored and aartamonau committed May 31, 2013
1 parent 6f3d379 commit 6f32b74
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion deps/ns_babysitter/src/ns_child_ports_sup.erl
Expand Up @@ -98,7 +98,7 @@ create_ns_server_supervisor_spec() ->
create_child_spec({Name, Cmd, Args, Opts}) ->
{{Name, Cmd, Args, Opts},
{supervisor_cushion, start_link,
[Name, 5000, 1000000, ns_port_server, start_link, [Name, Cmd, Args, Opts]]},
[Name, 5000, infinity, ns_port_server, start_link, [Name, Cmd, Args, Opts]]},
permanent, 86400000, worker,
[ns_port_server]}.

Expand Down

0 comments on commit 6f32b74

Please sign in to comment.