Nitsan Wakart

I added `ThreadUtil` in a package `org.jctools.util` the build fails: ``` package org.jctools.util; @InternalAPI public class ThreadUtil { public static void onSpinWait() { Thread.onSpinWait(); } } ``` Errors: ``` [ERROR]...

This patch needs to update https://github.com/JCTools/JCTools/blob/master/.travis.yml to use JDK 11 then.

I can confirm the code builds with `mvn clean install` but fails to build with just `mvn install`

On reflection, I think the right approach here would be a multi-module setup (which is also IDE friendly) with a build step packing the MR jar from all the module....

The `Map` javadoc is very confusing where `null` keys or values are concerned, and allows for optional throwing of NPE, which the ConcurrentMap default implementation should in theory be defensive...

OK, let's play JMM lawyers! :-P Here's `compute` javadoc: ``` * The default implementation is equivalent to performing the following * steps for this {@code map}, then returning the current...

Might be related, but AFAICT you are working under the assumption that compute is called once per successful insert, but the doc clearly suggests this is not the case("including potentially...

@adamretter I understand the usecase you are after, and I've read through the CHM implementation. There's no way I can see (though I only gave it 5 minutes of thinking,...

I'm not going to tackle this issue until JCTools drops JDK6/7 support.

Is this to build some bounded capacity XADD queue? I need to dig deeper into the xadd queues before having this conversation I suspect :-)