<div dir="ltr">If this actually yields a decrease in start time while not introducing other effects we would use it. Our start time of a couple minutes is annoying at times.</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 2, 2017 at 3:57 AM, Andrew Borodin <span dir="ltr"><<a href="mailto:borodin@octonica.com" target="_blank">borodin@octonica.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">2017-06-01 22:39 GMT+05:00 Maxim Dounin <<a href="mailto:mdounin@mdounin.ru">mdounin@mdounin.ru</a>>:<br>
> Thanks, though suggested change will certainly modify current<br>
> nginx (documented) approach of searching for minimum possible<br>
> hash sizes.<br>
><br>
> It might be a better solution for large hashes though, as<br>
> currently optimized by using a larger start size:<br>
><br>
>     if (hinit->max_size > 10000 && nelts && hinit->max_size / nelts < 100) {<br>
>         start = hinit->max_size - 1000;<br>
>     }<br>
<br>
</span>Yeah, maybe put the primes loop under if.<br>
<span class=""><br>
> Not sure it is at all needed though, as I don't remember any<br>
> "words about startup speed" in the documentation and startup<br>
> speed of hashes wasn't a practical problem as far as I remember.<br>
<br>
<br>
</span>Here  <a href="https://nginx.ru/en/docs/http/server_names.html" rel="noreferrer" target="_blank">https://nginx.ru/en/docs/http/<wbr>server_names.html</a><br>
"if nginx’s start time is unacceptably long, try to increase<br>
server_names_hash_bucket_size" It's about allowing more hash<br>
collisions.<br>
But I do not insist, though :) I've done the patch jut for fun<br>
<span class="im HOEnZb"><br>
Best regards, Andrey Borodin, Octonica.<br>
</span><div class="HOEnZb"><div class="h5">______________________________<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></div></div></blockquote></div><br></div>