<div dir="ltr">Hello, Maxim,<div><br></div><div>Thanks for your <span id="gmail-tran_2_21" class="gmail-" style="color:rgb(67,67,67);font-family:Arial,sans-serif;font-size:14px;background-color:rgb(252,252,254)">detailed</span><span style="color:rgb(67,67,67);font-family:Arial,sans-serif;font-size:14px;background-color:rgb(252,252,254)"> </span><span id="gmail-tran_2_22" class="gmail-" style="color:rgb(67,67,67);font-family:Arial,sans-serif;font-size:14px;background-color:rgb(252,252,254)">explanation and I truly agree with you.</span></div></div><div class="gmail_extra"><br><div class="gmail_quote">2017-06-19 23:28 GMT+08:00 Maxim Dounin <span dir="ltr"><<a href="mailto:mdounin@mdounin.ru" target="_blank">mdounin@mdounin.ru</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello!<br>
<span class=""><br>
On Mon, Jun 19, 2017 at 11:09:58PM +0800, Peng Fang wrote:<br>
<br>
> # HG changeset patch<br>
> # User RocFang <<a href="mailto:fangpeng1986@gmail.com">fangpeng1986@gmail.com</a>><br>
> # Date 1497882783 0<br>
> # Node ID 8b9e416ef7f9f8e7f96eaa53b47906<wbr>2683464481<br>
> # Parent  a39bc74873faf9e5bea616561b43f6<wbr>ecc55229f9<br>
> Introduced reload_delay.<br>
><br>
> Previously, the master process will sleep 100ms before sending a<br>
> SHUTDOWN signal to old worker processes when reload. This patch<br>
> make the sleep time configurable, because in some scenarios, the<br>
> new workers may spend more than 100ms to get ready. For example,<br>
> the init_prcess hook of some 3rd modules may be time-consuming.<br>
<br>
</span>The sleep in question is intended to let OS some time to actually<br>
start the process, and not intended to allow time-consuming work<br>
to happen on a worker process start.<br>
<br>
In general, no time-consuming operations are expected to be done<br>
during a worker process start.  Instead, time-consuming<br>
preparatory work is expected to happen in the context of the<br>
master process during configuration parsing and init module hooks.<br>
If a module does something time-consuming in the init process<br>
hook, it might be a good idea to change the module logic.<br>
<br>
Unless there is something more specific than a "the init_prcess<br>
hook of some 3rd modules may be time-consuming", I would rather<br>
reject the patch, as it introduces unneeded user-level complexity<br>
by adding a directive, and encourages bad module writing practice.<br>
<br>
Nevertheless, thank you for the patch.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Maxim Dounin<br>
<a href="http://nginx.org/" rel="noreferrer" target="_blank">http://nginx.org/</a><br>
______________________________<wbr>_________________<br>
nginx-devel mailing list<br>
<a href="mailto:nginx-devel@nginx.org">nginx-devel@nginx.org</a><br>
<a href="http://mailman.nginx.org/mailman/listinfo/nginx-devel" rel="noreferrer" target="_blank">http://mailman.nginx.org/<wbr>mailman/listinfo/nginx-devel</a><br>
</font></span></blockquote></div><br></div>