diff options
author | David Ansari <david.ansari@gmx.de> | 2023-03-20 14:46:45 +0000 |
---|---|---|
committer | David Ansari <david.ansari@gmx.de> | 2023-03-20 17:54:06 +0100 |
commit | 76db0b5bfacb59cca834f25c21e7cce52a3bdd7e (patch) | |
tree | 65e18bc3bb5a4eb121294be97c4de900f727f49e | |
parent | e3cf90a841cb827bcf287591aba29009c6aef01b (diff) | |
download | rabbitmq-server-git-76db0b5bfacb59cca834f25c21e7cce52a3bdd7e.tar.gz |
Make rabbit_fifo_dlx_sup more resilient
Previously, it used the default intensity:
"intensity defaults to 1 and period defaults to 5."
However, it's a bit low given there can be dozens or hundreds of DLX
workers: If only 2 fail within 5 seconds, the whole supervisor
terminates.
Even with the new values, there shouldn't be any infnite loop of the
supervisor terminating and restarting childs because the
rabbit_fifo_dlx_worker is terminated and started very quickly
given that the (the slow) consumer registration happens in
rabbit_fifo_dlx_worker:handle_continue/2.
-rw-r--r-- | deps/rabbit/src/rabbit_fifo_dlx_sup.erl | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/deps/rabbit/src/rabbit_fifo_dlx_sup.erl b/deps/rabbit/src/rabbit_fifo_dlx_sup.erl index f72040f8f2..4f0f4306b1 100644 --- a/deps/rabbit/src/rabbit_fifo_dlx_sup.erl +++ b/deps/rabbit/src/rabbit_fifo_dlx_sup.erl @@ -24,8 +24,8 @@ start_link() -> init([]) -> SupFlags = #{strategy => simple_one_for_one, - intensity => 1, - period => 5}, + intensity => 100, + period => 1}, Worker = rabbit_fifo_dlx_worker, ChildSpec = #{id => Worker, start => {Worker, start_link, []}, |