Github user mtaylor commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/1759#discussion_r160926081
--- Diff:
artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/ActiveMQServerImpl.java
---
@@ -2834,7 +2842,7 @@ public Queue createQueue(final SimpleString address,
throw
ActiveMQMessageBundle.BUNDLE.invalidRoutingTypeForAddress(routingType,
info.getName().toString(), info.getRoutingTypes());
}
- final QueueConfig queueConfig =
queueConfigBuilder.filter(filter).pagingManager(pagingManager).user(user).durable(durable).temporary(temporary).autoCreated(autoCreated).routingType(routingType).maxConsumers(maxConsumers).purgeOnNoConsumers(purgeOnNoConsumers).build();
+ final QueueConfig queueConfig =
queueConfigBuilder.filter(filter).pagingManager(pagingManager).user(user).durable(durable).temporary(temporary).autoCreated(autoCreated).routingType(routingTypeToUse).maxConsumers(maxConsumers).purgeOnNoConsumers(purgeOnNoConsumers).build();
--- End diff --
The routing type should be decided by the protocol handler and passed in.
---
