AntonRoskvist opened a new pull request, #5296:
URL: https://github.com/apache/activemq-artemis/pull/5296

   I got interested by #3479 and decided to test those changes out against a 
"fresh" build of the broker. I found it quite impressive and decided to 
implement what was discussed on @franz1981 original pr in hopes it would 
interest someone else. I take no credit for this PR, as it contains almost 
exclusively the original changes and what was discussed in #3479.
    
   One thing I did actually change was to default the amount of 
"remotingThreads" to 1 instead of "3 * CPUCount" (for IO_URING) based on 
suggestions in the original PR as well as here: 
[netty-incubator-transport-io_uring/issues/106#issuecomment-876173958]( 
https://github.com/netty/netty-incubator-transport-io_uring/issues/106#issuecomment-876173958)
   
   According to my own testing this is what the default value should be, with 
only a few "extremes"  benefiting from a few more.
   
   I have seen some impressive results from using IO_URING as the transport 
that I should be able to share after a few more rounds of testing.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: gitbox-unsubscr...@activemq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@activemq.apache.org
For additional commands, e-mail: gitbox-h...@activemq.apache.org
For further information, visit: https://activemq.apache.org/contact


Reply via email to