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

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

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

    https://github.com/apache/activemq-artemis/pull/672#discussion_r72421069
  
    --- Diff: 
artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueImpl.java
 ---
    @@ -2983,6 +2988,18 @@ else if (consumerRate < threshold) {
                       if (connection != null) {
                          
ActiveMQServerLogger.LOGGER.slowConsumerDetected(serverConsumer.getSessionID(), 
serverConsumer.getID(), getName().toString(), connection.getRemoteAddress(), 
threshold, consumerRate);
                          if (policy.equals(SlowConsumerPolicy.KILL)) {
    +                        if (connection instanceof CoreRemotingConnection) {
    +                           CoreRemotingConnection coreconnection = 
(CoreRemotingConnection)connection;
    +
    +                           Channel channel1 = coreconnection.getChannel(1, 
-1);
    --- End diff --
    
    maybe `clientChannel` or something similar? `channel1` doesn't give much 
description.


> Disable slow client reconnecting with KILL slow client policy
> -------------------------------------------------------------
>
>                 Key: ARTEMIS-642
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-642
>             Project: ActiveMQ Artemis
>          Issue Type: New Feature
>    Affects Versions: 1.3.0
>            Reporter: Chen Maoqian
>            Priority: Minor
>
> It is possible to detect and handle(KILL, NOTIFY) slow consumers. In case 
> policy KILL is used and connection factory used to create connection of slow 
> consumer has attribute reconnect-attempts set to "-1", slow consumer 
> reconnects to server. This consumer is then reconnects, is detected as slow 
> and disconnected. 
> Attribute reconnect-attempts="-1" should be ignored, and slow client should 
> not be trying to reconnect once it is disconnected by KILL slow consumer 
> policy.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to