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