[GitHub] activemq-artemis pull request #2191: ARTEMIS-1987 - Add consumer window size...

[michaelandrepearce]

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

    https://github.com/apache/activemq-artemis/pull/2191#discussion_r204784325
  
    --- Diff: 
artemis-core-client/src/main/java/org/apache/activemq/artemis/core/client/impl/ServerLocatorImpl.java
 ---
    @@ -347,7 +347,7 @@ private ServerLocatorImpl(final Topology topology,
     
           minLargeMessageSize = ActiveMQClient.DEFAULT_MIN_LARGE_MESSAGE_SIZE;
     
    -      consumerWindowSize = ActiveMQClient.DEFAULT_CONSUMER_WINDOW_SIZE;
    +      consumerWindowSize = -1;
    --- End diff --
    
    Should still be defaulted to a constant


---