[ARTEMIS-859] BACKLOG_PROP_NAME is not part of allowable properties on the acceptor

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.5.0
• Fix Version/s: 1.5.1
• Assignee: Clebert Suconic
• Reporter: Shaun Appleton
• Created: 18/Nov/16 16:14
• Updated: 21/Nov/16 16:28
• Resolved: 21/Nov/16 16:22

Description

org/apache/arctivemq/artemis/core/remoting/impl/netty/TransportConstants.java uses a hashmap named allowableAcceptorKeys which does not allow the BACKLOG_PROP_NAME to be set, even though it is there.

The change is to add the line
allowableAcceptorKeys.add(TransportConstants.BACKLOG_PROP_NAME);

Changing the netty acceptors to include backlog is also needed as follows

<acceptors>
<netty-acceptor name="netty" socket-binding="messaging">
<param key="use-nio" value="true"/>
<param key="backlog" value="1028"/>
</netty-acceptor>
<netty-acceptor name="netty-throughput" socket-binding="messaging-throughput">
<param key="batch-delay" value="50"/>
<param key="direct-deliver" value="false"/>
<param key="use-nio" value="true"/>
<param key="backlog" value="1028"/>
</netty-acceptor>
</acceptors>

Issue Links

• links to: GitHub Pull Request #893