activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jbertram <...@git.apache.org>
Subject [GitHub] activemq-artemis pull request #1887: ARTEMIS-1696 allow empty 'address' for ...
Date Fri, 23 Feb 2018 02:40:54 GMT
Github user jbertram commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/1887#discussion_r170153460
  
    --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/deployers/impl/FileConfigurationParser.java
---
    @@ -1671,7 +1671,7 @@ private void parseClusterConnectionConfiguration(final Element e,
final Configur
        private void parseGroupingHandlerConfiguration(final Element node, final Configuration
mainConfiguration) {
           String name = node.getAttribute("name");
           String type = getString(node, "type", null, Validators.NOT_NULL_OR_EMPTY);
    -      String address = getString(node, "address", null, Validators.NOT_NULL_OR_EMPTY);
    +      String address = getString(node, "address", "", Validators.NO_CHECK);
    --- End diff --
    
    I did that to match the change that was made for the cluster-connection address parsing.
 See https://github.com/apache/activemq-artemis/blob/master/artemis-server/src/main/java/org/apache/activemq/artemis/core/deployers/impl/FileConfigurationParser.java#L1581.


---

Mime
View raw message