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

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_r72420954
  
    --- Diff: 
artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueImpl.java
 ---
    @@ -2971,6 +2975,7 @@ public void run() {
                    else if (consumerRate < threshold) {
                       RemotingConnection connection = null;
                       RemotingService remotingService = ((PostOfficeImpl) 
postOffice).getServer().getRemotingService();
    +                  ActiveMQServer server = ((PostOfficeImpl) 
postOffice).getServer();
    --- End diff --
    
    The code now casts twice.  Should this be in the interface? Should the 
casting be done once?  Can server be extracted for the line above?


> 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