<div dir="ltr">Hello,<div><br></div><div>Any news regarding this patch?</div></div><div class="gmail_extra"><br><div class="gmail_quote">2014-06-27 11:42 GMT+03:00 Francesco Lattanzio <span dir="ltr"><<a href="mailto:franz.lattanzio@gmail.com" target="_blank">franz.lattanzio@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Removed dots from commit messages.<br>
A new pull request has been submitted. Here is the new patch:<br>
<br>
  git fetch <a href="https://github.com/fltt/otp.git" target="_blank">https://github.com/fltt/otp.git</a> ets_nested_lht_4<br>
<br>
  <a href="https://github.com/fltt/otp/compare/erlang:maint...ets_nested_lht_4" target="_blank">https://github.com/fltt/otp/compare/erlang:maint...ets_nested_lht_4</a><br>
  <a href="https://github.com/fltt/otp/compare/erlang:maint...ets_nested_lht_4.patch" target="_blank">https://github.com/fltt/otp/compare/erlang:maint...ets_nested_lht_4.patch</a><br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Francesco Lattanzio<br>
</font></span><span class="im HOEnZb"><br>
On Wed, Jun 25, 2014 at 09:38:06PM +0200, Francesco Lattanzio wrote:<br>
> Hello again Sverker,<br>
>     I've made some progress with this patch. Briefly:<br>
><br>
> * set tables and bag/duplicate_bag tables are now managed by dedicated<br>
>   code -- erl_db_hash.[ch] and erl_db_nested_hash.[ch] respectively<br>
> * reduced memory overhead<br>
><br>
> You can compute the difference (in words) between the new and the old<br>
> implementations' memory footprints (when nested LHT is not active) of a<br>
> collection of 'n' objects sharing the same key as:<br>
><br>
>   MemDiff = 4 - n<br>
><br>
> I used this formula to fix the ets suite's memory test case.<br>
><br>
> With regard to the yielding problem, I deciced not to fix it now for the<br>
> following reasons:<br>
><br>
> 1) it is unrelated to the nested LHT, as the problem manifests itself<br>
>    even in the current (non-nested LHT) implementation<br>
> 2) you offered to fix it yourself<br>
><br>
> The patch is available at:<br>
><br>
>   git fetch <a href="https://github.com/fltt/otp.git" target="_blank">https://github.com/fltt/otp.git</a> ets_nested_lht_3<br>
><br>
>   <a href="https://github.com/fltt/otp/compare/erlang:maint...ets_nested_lht_3" target="_blank">https://github.com/fltt/otp/compare/erlang:maint...ets_nested_lht_3</a><br>
>   <a href="https://github.com/fltt/otp/compare/erlang:maint...ets_nested_lht_3.patch" target="_blank">https://github.com/fltt/otp/compare/erlang:maint...ets_nested_lht_3.patch</a><br>
><br>
> It is based on the maint branch.<br>
><br>
> I've tested it on Linux x86_64, latest kernel (all the test suites) and<br>
> FreeBSD/i386 9.2 (only the emulator, kernel and stdlib test suites).<br>
><br>
> Have another nice day.<br>
</span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
erlang-patches mailing list<br>
<a href="mailto:erlang-patches@erlang.org">erlang-patches@erlang.org</a><br>
<a href="http://erlang.org/mailman/listinfo/erlang-patches" target="_blank">http://erlang.org/mailman/listinfo/erlang-patches</a><br>
</div></div></blockquote></div><br></div>