<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">

<html>

  <head>

    <meta content="text/html; charset=ISO-8859-1"

      http-equiv="Content-Type">

  </head>

  <body bgcolor="#ffffff" text="#000000">

    On 09/14/2011 06:00 PM, Christopher Faulet wrote:

    <blockquote cite="mid:4E70CF83.6060207@capflam.org" type="cite">

      <pre wrap="">Le 14/09/2011 11:51, Siri Hansen a écrit :

</pre>

      <blockquote type="cite">

        <pre wrap="">I have discussed the simple_one_for_one_shutdown problem with my

colleagues, and finally we came to the same conclusion as you did :) The

current behavior is definitely not acceptable so we decided to take in

you patch. If you don't mind I would however like you to do some minor

adjustments first:

1) I think it would be good to document the differences in how shutdown

is handled for simple_one_for_one supervisors compared to other

supervisors. I.e. that there is no defined shutdown order, and that the

workers will terminate "in parallel".

2) If the shutdown strategy for the dynamic children is a timer, and the

timer expires - then it looks like there will be one error report

printed per worker that is not yet terminated (please correct me if I'm

wrong). I believe that this could be quite heavy if there are many

workers left. I would suggest to print only one error report in this

situation - either listing all pids or just say how many processes we

are talking about.

Also, as Henrik writes, your supervisor_shutdown_infinity patch is

already included in pu. Before finally including it in our development

branch, I would also like you to add some more information to the

documentation (a warning?) - about using this feature with care, and if

you need to use it then implement the worker in a safe way so the

cleanup procedure can not hang... since this can actually cause the

termination of the complete supervisor tree to hang.

</pre>

      </blockquote>

      <pre wrap="">

Siri,

Thanks for your feedback. You can re-fetch the branches:

git fetch git://github.com/capflam/otp.git simple_one_for_one_shutdown

git fetch git://github.com/capflam/otp.git supervisor_shutdown_infinity

Let me know if there is any problem with my updates (Especially with the

documentation).

</pre>

      <pre wrap="">

<fieldset class="mimeAttachmentHeader"></fieldset>

_______________________________________________

erlang-patches mailing list

<a class="moz-txt-link-abbreviated" href="mailto:erlang-patches@erlang.org">erlang-patches@erlang.org</a>

<a class="moz-txt-link-freetext" href="http://erlang.org/mailman/listinfo/erlang-patches">http://erlang.org/mailman/listinfo/erlang-patches</a>

</pre>

    </blockquote>

    Refetched supervisor_shutdown_infinity

    <br>

    <br>

    Included simple_one_for_one_shutdown

    in 'pu'<br>

    <br>

    Thank you<br>

    <pre class="moz-signature" cols="72">-- 

/Henrik Nord Erlang/OTP</pre>

  </body>

</html>