franz1981 edited a comment on issue #2524: ARTEMIS-2240 
ActiveMQThreadPoolExecutor should use LinkedTransferQueue
URL: https://github.com/apache/activemq-artemis/pull/2524#issuecomment-457884054
 
 
   On my box I'm getting the same performance of the master version for single 
producer/single consumer workloads and a better scalability with 8 producers/8 
consumers (on different queues) for both the commits I've attached (the second 
one, in particular).
   I'm putting this for review and for validation: if anyone has a better 
machine to validate its behaviour is would be great. On executor benchmarks (in 
particular the second commit) it is much better then the original solution, but 
benchmarks are...just benchmarks: need to check inside the broker mechanics and 
workloads.
   
   A conservative approach will suggest to just use the first commit ie a 
simple q replacement, but I the second commit has more to offer given that 
would allow to use what this q offer out of the box and that seems very similar 
to what the thread pool was delivering in the implementation on master.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to