[jira] [Work logged] (ARTEMIS-3137) Support XPath filters

["ASF GitHub Bot (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/ARTEMIS-3137?focusedWorklogId=559335&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-559335 ]

ASF GitHub Bot logged work on ARTEMIS-3137:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 01/Mar/21 11:51
            Start Date: 01/Mar/21 11:51
    Worklog Time Spent: 10m 
      Work Description: gemmellr commented on a change in pull request #3466:
URL: https://github.com/apache/activemq-artemis/pull/3466#discussion_r584650747



##########
File path: artemis-selector/src/main/java/org/apache/activemq/artemis/selector/filter/XPathExpression.java
##########
@@ -79,4 +97,15 @@ public boolean matches(Filterable message) throws FilterException {
       return object == Boolean.TRUE;
    }
 
+   protected static void setupFeatures(DocumentBuilderFactory factory) throws ParserConfigurationException {
+      Properties properties = System.getProperties();
+      for (Map.Entry<Object, Object> prop : properties.entrySet()) {
+         String key = (String) prop.getKey();
+         if (key.startsWith(DOCUMENT_BUILDER_FACTORY_FEATURE)) {
+            String uri = key.split(DOCUMENT_BUILDER_FACTORY_FEATURE + ":")[1];

Review comment:
       Sure, but adding stuff to a fresh codebase is a good time to actually tidy it up and fix things. Rather than introducing it as-is, meaning it likely never happens and folks have to see it and/or possibly hit it later. Essentially, if a trivial bug/improvement is easily spotted in review, and people are being asked to review...




----------------------------------------------------------------
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.

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


Issue Time Tracking
-------------------

    Worklog Id:     (was: 559335)
    Time Spent: 4h 10m  (was: 4h)

> Support XPath filters
> ---------------------
>
>                 Key: ARTEMIS-3137
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-3137
>             Project: ActiveMQ Artemis
>          Issue Type: New Feature
>            Reporter: Justin Bertram
>            Assignee: Justin Bertram
>            Priority: Major
>          Time Spent: 4h 10m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)