jbertram opened a new pull request, #5757:
URL: https://github.com/apache/activemq-artemis/pull/5757

   This commit replaces the matching logic for MQTT subscription IDs which uses 
regular expression for a bit of pre-existing logic that I didn't know about 
when this functionality was first implemented.
   
   This saves the expense of generating an instance of 
`java.util.regex.Pattern` and uses basic `SimpleString` or `char` comparisons 
which are optimized for the wildcard address use-case.
   
   This also removes the `createPattern` method from `o.a.a.a.c.s.i.Match` 
since the only code using that method has now also been removed. Related tests 
have also been removed.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: gitbox-unsubscr...@activemq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@activemq.apache.org
For additional commands, e-mail: gitbox-h...@activemq.apache.org
For further information, visit: https://activemq.apache.org/contact


Reply via email to