[GitHub] activemq-artemis pull request #2237: [ARTEMIS-2022] Create count messages 'g...

[michaelandrepearce]

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

    https://github.com/apache/activemq-artemis/pull/2237#discussion_r209381476
  
    --- Diff: 
artemis-server/src/main/java/org/apache/activemq/artemis/core/management/impl/QueueControlImpl.java
 ---
    @@ -711,6 +712,32 @@ public long countMessages(final String filterStr) 
throws Exception {
           }
        }
     
    +   @Override
    +   public String countMessagesProperty(final String filter) throws 
Exception {
    +      checkStarted();
    +      clearIO();
    +      try {
    +         try (LinkedListIterator<MessageReference> iterator = 
queue.browserIterator()) {
    +            Map<String, Integer> result = new HashMap<>();
    --- End diff --
    
    Counter should be long, not integer


---