[GitHub] activemq-artemis pull request #2243: Artemis-2022 create count messages with...

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

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


---


[GitHub] activemq-artemis pull request #2243: Artemis-2022 create count messages with...

2018-08-13 Thread michaelandrepearce
GitHub user michaelandrepearce opened a pull request:

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

Artemis-2022 create count messages with group by filter



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

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

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

https://github.com/apache/activemq-artemis/pull/2243.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 #2243


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

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

commit d7d7f7499cf7ca675f901090e3199ad71bc0f922
Author: Arthur Fritz Santiago 
Date:   2018-08-10T15:09:32Z

[ARTEMIS-2022] Ajust checkstyle

commit fdf3fadac52b94f060e8baff3e475c9f806b4baa
Author: Michael André Pearce 
Date:   2018-08-10T21:37:26Z

ARTEMIS-2022 - Enhancements 

Fix checkstyle
Avoid duplicated logic
Ability to filter and group
Instantiate SimpleString property key once
Get property value via getObjectProprty to ensure all special mapped 
properties such as in AMQPMessage would return
Avoid a custom string to represent null, instead rely on Java's 
representation "null" by using Objects.toString to get the string value of the 
property value used to group by.




---