[ 
https://issues.apache.org/jira/browse/ARTEMIS-1420?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16170947#comment-16170947
 ] 

ASF GitHub Bot commented on ARTEMIS-1420:
-----------------------------------------

Github user michaelandrepearce commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/1534#discussion_r139573407
  
    --- Diff: 
tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/amqp/AmqpClientTestSupport.java
 ---
    @@ -185,6 +187,7 @@ protected TransportConfiguration 
addAcceptorConfiguration(ActiveMQServer server,
           HashMap<String, Object> params = new HashMap<>();
           params.put(TransportConstants.PORT_PROP_NAME, String.valueOf(port));
           params.put(TransportConstants.PROTOCOLS_PROP_NAME, 
getConfiguredProtocols());
    +      params.put(TransportConstants.NETTY_READ_TIMEOUT, readTimeout);
    --- End diff --
    
    3 ms seems very small even for test case, also test should test without 
setting this that error does occur. Eg assert the problem exists without 
setting.


> limit non-ssl connection hangs up exceeding client(s)
> -----------------------------------------------------
>
>                 Key: ARTEMIS-1420
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-1420
>             Project: ActiveMQ Artemis
>          Issue Type: Improvement
>    Affects Versions: 2.3.0
>            Reporter: Stanislav Knot
>
> this is a max-negotiate-time. Maximum time a connection can take to send the 
> initial protocol negotiation, so time between creating tcp connection and 
> sending the first frame.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to