<p dir="ltr">Why not. Only doubt I&#39;d have is that other usages of the CHM are - I guess - services registry and similar configuration tools, for which write performance is irrelevant: your test measured puts, are there drawbacks on gets or memory usage? </p>

<p dir="ltr">Recently you changed all (most?) CHM creations to use a consistent factory, maybe we could improve on that by actually using a couple of factories which differentiate on the intended usage of the CHM: for example some maps who change very infrequently - mostly during boot or reconfiguration, maybe even topology change - could be better served by a non concurrent structure using copy-on-wrtite. </p>

<p dir="ltr">Sanne </p>
<div class="gmail_quote">On 19 Apr 2013 08:48, &quot;Dan Berindei&quot; &lt;<a href="mailto:dan.berindei@gmail.com">dan.berindei@gmail.com</a>&gt; wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div>+1 to make CHMv8 the default on JDK6 and JDK7<br><br></div><div>But I&#39;m not convinced we should make it the default for JDK8 - even though we don&#39;t know exactly what we&#39;re getting with the JDK&#39;s implementation.<br>


</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Apr 19, 2013 at 5:39 AM, David M. Lloyd <span dir="ltr">&lt;<a href="mailto:david.lloyd@redhat.com" target="_blank">david.lloyd@redhat.com</a>&gt;</span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>On 04/18/2013 09:35 PM, Manik Surtani wrote:<br>
&gt; Guys,<br>
&gt;<br>
&gt; Based on some recent micro benchmarks I&#39;ve been doing, I&#39;ve seen:<br>
&gt;<br>
&gt; MapStressTest configuration: capacity 100000, test running time 60 seconds<br>
&gt; Testing mixed read/write performance with capacity 100,000, keys 300,000, concurrency level 32, threads 12, read:write ratio 0:1<br>
&gt; Container CHM           Ops/s 21,165,771.67  Gets/s       0.00  Puts/s 21,165,771.67  HitRatio     100.00  Size    262,682  stdDev 77,540.73<br>
&gt; Container CHMV8         Ops/s 33,513,807.09  Gets/s       0.00  Puts/s 33,513,807.09  HitRatio     100.00  Size    262,682  stdDev 77,540.73<br>
&gt;<br>
&gt; So under high concurrency (12 threads, on my workstation with 12 hardware threads - so all threads are always working), we see that Infinispan&#39;s CHMv8 implementation is 50% faster than JDK6&#39;s CHM implementation when doing puts.<br>



&gt;<br>
&gt; We use a fair number of CHMs all over Infinispan&#39;s codebase.  By default, these are all JDK-provided CHMs.  But we have the option to switch to our CHMv8 implementation by passing in -Dinfinispan.unsafe.allow_jdk8_chm=true.<br>



&gt;<br>
&gt; The question is, should this be the default?  Thoughts, opinions?<br>
<br>
</div>The JDK&#39;s concurrency code - especially CHM - changes all the time.<br>
You&#39;d be very well-served, in my opinion, to go with something like<br>
CHMv8 just because you could be so much more sure that you&#39;ll have more<br>
consistent (and possibly better, but definitely more consistent)<br>
performance across all JVMs, instead of being at the mercy of whatever<br>
particular implementation happens to run on whatever JVM.<br>
<span><font color="#888888"><br>
<br>
--<br>
- DML<br>
</font></span><div><div>_______________________________________________<br>
infinispan-dev mailing list<br>
<a href="mailto:infinispan-dev@lists.jboss.org" target="_blank">infinispan-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/infinispan-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/infinispan-dev</a><br>
</div></div></blockquote></div><br></div>
<br>_______________________________________________<br>
infinispan-dev mailing list<br>
<a href="mailto:infinispan-dev@lists.jboss.org">infinispan-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/infinispan-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/infinispan-dev</a><br></blockquote></div>