[GitHub] activemq-artemis issue #2399: ARTEMIS-2150 Counts the number of delivering m...

2018-10-26 Thread ArthurFritz
Github user ArthurFritz commented on the issue:

https://github.com/apache/activemq-artemis/pull/2399
  
@michaelandrepearce Nice, Tks


---


[GitHub] activemq-artemis issue #2398: ARTEMIS-2150 Counts the number of delivering m...

2018-10-26 Thread ArthurFritz
Github user ArthurFritz commented on the issue:

https://github.com/apache/activemq-artemis/pull/2398
  
Okay, i created new Merge Request, adjusted commit message :smiley: 


---


[GitHub] activemq-artemis pull request #2399: ARTEMIS-2150 Counts the number of deliv...

2018-10-26 Thread ArthurFritz
GitHub user ArthurFritz opened a pull request:

https://github.com/apache/activemq-artemis/pull/2399

ARTEMIS-2150 Counts the number of delivering messages in this queue

Create function this counts the number of delivering messages in this queue 
matching the specified filter.

Create a other function this counts the number of delivering messages in 
this queue matching the specified filter, grouped by the given property field.

[JIRA](https://issues.apache.org/jira/browse/ARTEMIS-2150)

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/ArthurFritz/activemq-artemis 
feature/ARTEMIS-2150

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/activemq-artemis/pull/2399.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2399


commit 5b32235d7ca4230bfb6d4956ffdef026d5bb51cb
Author: Arthur Fritz Santiago 
Date:   2018-10-26T18:32:34Z

ARTEMIS-2150 Counts the number of delivering messages in this queue




---


[GitHub] activemq-artemis pull request #2398: ARTEMIS-2150 Counts the number of deliv...

2018-10-26 Thread ArthurFritz
Github user ArthurFritz closed the pull request at:

https://github.com/apache/activemq-artemis/pull/2398


---


[GitHub] activemq-artemis pull request #2398: [ARTEMIS-2150] Counts the number of del...

2018-10-26 Thread ArthurFritz
GitHub user ArthurFritz opened a pull request:

https://github.com/apache/activemq-artemis/pull/2398

[ARTEMIS-2150] Counts the number of delivering messages in this queue

Create function this counts the number of delivering messages in this queue 
matching the specified filter.

Create a other function this counts the number of delivering messages in 
this queue matching the specified filter, grouped by the given property field.

[JIRA](https://issues.apache.org/jira/browse/ARTEMIS-2150)

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/ArthurFritz/activemq-artemis 
feature/ARTEMIS-2150

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/activemq-artemis/pull/2398.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2398


commit 1396ec6d825708994fa8d680f5c42480aacd410a
Author: Arthur Fritz Santiago 
Date:   2018-10-26T17:39:09Z

[ARTEMIS-2150] Counts the number of delivering messages in this queue




---


[GitHub] activemq-artemis issue #2237: [ARTEMIS-2022] Create count messages 'group by...

2018-08-13 Thread ArthurFritz
Github user ArthurFritz commented on the issue:

https://github.com/apache/activemq-artemis/pull/2237
  
Cool, is there an expected date for 2.7.0?


---


[GitHub] activemq-artemis issue #2237: [ARTEMIS-2022] Create count messages 'group by...

2018-08-13 Thread ArthurFritz
Github user ArthurFritz commented on the issue:

https://github.com/apache/activemq-artemis/pull/2237
  
@michaelandrepearce You're welcome.
Would you know the expected date of version 2.6.3 ?


---


[GitHub] activemq-artemis issue #2237: [ARTEMIS-2022] Create count messages 'group by...

2018-08-13 Thread ArthurFritz
Github user ArthurFritz commented on the issue:

https://github.com/apache/activemq-artemis/pull/2237
  
Ok, thanks.


---


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

2018-08-13 Thread ArthurFritz
Github user ArthurFritz closed the pull request at:

https://github.com/apache/activemq-artemis/pull/2237


---


[GitHub] activemq-artemis issue #2237: [ARTEMIS-2022] Create count messages 'group by...

2018-08-11 Thread ArthurFritz
Github user ArthurFritz commented on the issue:

https://github.com/apache/activemq-artemis/pull/2237
  
Ok, nice idea include filter, and selector to group messages.


---


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

2018-08-10 Thread ArthurFritz
Github user ArthurFritz commented on a diff in the pull request:

https://github.com/apache/activemq-artemis/pull/2237#discussion_r209317397
  
--- 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 iterator = 
queue.browserIterator()) {
+Map result = new HashMap<>();
+String propertySearch = filter == null ? UNDEFINED : filter;
+try {
+   while (iterator.hasNext()) {
+  MessageReference ref = iterator.next();
+  String messageProperty = 
ref.getMessage().getStringProperty(propertySearch);
--- End diff --


It may be an object, however its toString will be used


---


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

2018-08-10 Thread ArthurFritz
Github user ArthurFritz commented on a diff in the pull request:

https://github.com/apache/activemq-artemis/pull/2237#discussion_r209317077
  
--- 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 {
--- End diff --


There are already counters per filter countMessages (), in this case it 
would be more a "group by", without filter, always counting the messages of the 
property.


---


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

2018-08-10 Thread ArthurFritz
Github user ArthurFritz commented on a diff in the pull request:

https://github.com/apache/activemq-artemis/pull/2237#discussion_r209314996
  
--- 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 {
--- End diff --


In fact it will not filter the messages, but will count the messages for 
your property.
In a project I have several properties that I use in selectors, this count 
will show me how many messages there are per property variation.


---


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

2018-08-10 Thread ArthurFritz
GitHub user ArthurFritz opened a pull request:

https://github.com/apache/activemq-artemis/pull/2237

[ARTEMIS-2022] Create count messages 'group by' this property filter

Create a new function this is responsibility count messages determined 
property filter, e return one object in json, where properties to object is 
value of property and this value is amount messages.
Actually in the project have only method count messages of determinate 
filter, but no has method group messages for respective property.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/ArthurFritz/activemq-artemis 
feature/ARTEMIS-2022

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/activemq-artemis/pull/2237.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2237


commit 259f3f8be9014b79503e1ed0ead8c9598f340ff6
Author: Arthur Fritz Santiago 
Date:   2018-08-10T14:49:16Z

[ARTEMIS-2022] Create count messages 'group by' this property filter




---


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

2018-08-10 Thread ArthurFritz
Github user ArthurFritz closed the pull request at:

https://github.com/apache/activemq-artemis/pull/2236


---


[GitHub] activemq-artemis issue #2236: [ARTEMIS-2022] Create count messages 'group by...

2018-08-10 Thread ArthurFritz
Github user ArthurFritz commented on the issue:

https://github.com/apache/activemq-artemis/pull/2236
  
Alter destination feature.


---


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

2018-08-10 Thread ArthurFritz
GitHub user ArthurFritz opened a pull request:

https://github.com/apache/activemq-artemis/pull/2236

[ARTEMIS-2022] Create count messages 'group by' this property filter

Create a new function this is responsibility count messages determined 
property filter, e return one object in json, where properties to object is 
value of property and this value is amount messages.
Actually in the project have only method count messages of determinate 
filter, but no has method group messages for respective property.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/ArthurFritz/activemq-artemis 
feature/ARTEMIS-2022

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/activemq-artemis/pull/2236.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2236


commit d758bc01f6082d0a9927c350d443c6a9abaf247d
Author: Arthur Fritz Santiago 
Date:   2018-08-10T14:13:44Z

[ARTEMIS-2022] Create count messages 'group by' this property filter




---