[GitHub] activemq-artemis pull request #2200: ARTEMIS-1997 - un-needed SimpleString c...

2018-07-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/2200 ---

[GitHub] activemq-artemis pull request #2200: ARTEMIS-1997 - un-needed SimpleString c...

2018-07-30 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2200#discussion_r206141050 --- Diff: artemis-selector/src/main/java/org/apache/activemq/artemis/selector/filter/Filterable.java --- @@ -41,7 +43,7 @@ * @param

[GitHub] activemq-artemis pull request #2200: ARTEMIS-1997 - un-needed SimpleString c...

2018-07-30 Thread michaelandrepearce
GitHub user michaelandrepearce opened a pull request: https://github.com/apache/activemq-artemis/pull/2200 ARTEMIS-1997 - un-needed SimpleString creation on hotpath with Filters Create the SimpleString on construction of PropertyExpression so it can be re-used instead of creating