Make the number of Netty worker threads number configurable

[komamitsu]

I noticed Moquette users can't configure the number of Netty worker threads in current Moquette. Increasing Netty worker threads number is important in our use case that handles a lot of MQTT requests.

This PR makes it configurable.

[andsel]

This is not necessary. Netty used asunch IO an Moquette is essentially IO bound so adding thread only adds threads that do IO waiting. The broker starts and Co figure the number of threads equals to the number of cores - 1, let this configurable to put for example 1000 threads it's a no sense, just adds weight to the scheduler and stole memory for each thread's stack. Il lun 8 apr 2019, 07:24 Mitsunori Komatsu <notifications@github.com> ha scritto:

…

I noticed Moquette users can't configure the number of Netty worker threads in current Moquette. Increasing Netty worker threads number is important in our use case that handles a lot of MQTT requests. This PR makes it configurable. ------------------------------ You can view, comment on, or merge this pull request online at: #468 Commit Summary - Add BrokerConstants.NETTY_WORKER_THREADS File Changes - *M* broker/src/main/java/io/moquette/BrokerConstants.java <https://github.com/moquette-io/moquette/pull/468/files#diff-0> (1) - *M* broker/src/main/java/io/moquette/broker/NewNettyAcceptor.java <https://github.com/moquette-io/moquette/pull/468/files#diff-1> (13) Patch Links: - https://github.com/moquette-io/moquette/pull/468.patch - https://github.com/moquette-io/moquette/pull/468.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#468>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AA_Y0dOkYC03U8T_1P3ARPjffNKz8uxPks5vetKKgaJpZM4chUpc> .