[GitHub] qpid-jms issue #26: QPIDJMS-430 Lock-Free FifoMessageQueue

2018-11-14 Thread franz1981
Github user franz1981 commented on the issue: https://github.com/apache/qpid-jms/pull/26 I have TBH that I'm not getting such a big boost as I was expecting, but probably because there are other bottlenecks (the consumer side on the broker) that are not' helping to measure it

[GitHub] qpid-jms issue #26: QPIDJMS-430 Lock-Free FifoMessageQueue

2018-11-14 Thread tabish121
Github user tabish121 commented on the issue: https://github.com/apache/qpid-jms/pull/26 Yes, that test is exposing some issues in the 5.x broker which is unrelated to this change. --- - To unsubscribe, e-mail:

[GitHub] qpid-jms issue #26: QPIDJMS-430 Lock-Free FifoMessageQueue

2018-11-14 Thread franz1981
Github user franz1981 commented on the issue: https://github.com/apache/qpid-jms/pull/26 The failing test `JmsConnectionCloseVariationsTest.testCloseBeforeBrokerStoppedRepeated` seems that is not allocating any `FifoMessageQueue`: i suppose its failure is indipendent by this PR.

[GitHub] qpid-jms issue #26: QPIDJMS-430 Lock-Free FifoMessageQueue

2018-11-14 Thread franz1981
Github user franz1981 commented on the issue: https://github.com/apache/qpid-jms/pull/26 Just as a reference: I'm getting about 80 M msg/sec with the new `FifoMessageQueue` while near 2.5 M msg/sec with the original one. On a end-to-end test I'm getting a 20% more throughput with

[GitHub] qpid-jms issue #26: QPIDJMS-430 Lock-Free FifoMessageQueue

2018-11-14 Thread jdanekrh
Github user jdanekrh commented on the issue: https://github.com/apache/qpid-jms/pull/26 Won't work with JDK < 8 is fine with qpid-jms, I think https://github.com/apache/qpid-jms/blob/master/pom.xml#L37-L39 I'll check tomorrow if IBM JDK supports this. I am fairly sure

[GitHub] qpid-jms issue #26: QPIDJMS-430 Lock-Free FifoMessageQueue

2018-11-14 Thread franz1981
Github user franz1981 commented on the issue: https://github.com/apache/qpid-jms/pull/26 It won't work with JDK < 8 and is specific of oracle AFAIK (I could be wrong of course!) --- - To unsubscribe, e-mail:

[GitHub] qpid-jms issue #26: QPIDJMS-430 Lock-Free FifoMessageQueue

2018-11-14 Thread jdanekrh
Github user jdanekrh commented on the issue: https://github.com/apache/qpid-jms/pull/26 Wouldn't `@Contended` annotation work just as well? http://mail.openjdk.java.net/pipermail/hotspot-dev/2012-November/007309.html (I just googled it...) ---

[GitHub] qpid-jms issue #26: QPIDJMS-430 Lock-Free FifoMessageQueue

2018-11-14 Thread franz1981
Github user franz1981 commented on the issue: https://github.com/apache/qpid-jms/pull/26 The reason behind the abstract classes for padding between fields to avoid false sharing (that could lead to a 1/10 of performance): ``` OFFSET SIZE