ASF GitHub Bot logged work on ARTEMIS-2680:

                Author: ASF GitHub Bot
            Created on: 25/Mar/20 22:41
            Start Date: 25/Mar/20 22:41
    Worklog Time Spent: 10m 
      Work Description: michaelpearce-gain commented on issue #3047: 
ARTEMIS-2680 use q name for addr if none specified via config
URL: https://github.com/apache/activemq-artemis/pull/3047#issuecomment-604124890
   isnt the overloaded createQueue a legacy of pre 2.x model, where queues and 
topics were modelled differently and thus to keep some compatibilty and for JMS 
server bits that were still about, the createQueue was more for JMS create a 
JMS queue where queue and address did match.
   Now the question i would ask, is should you be able to make a queue without 
an address, to me it seems we shouldnt so if anything we should probably clean 
out that createQueue method that creates and address based on queueName , 
createQueue really must/should provide an address IMO, and error if not.
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:

Issue Time Tracking

    Worklog Id:     (was: 409945)
    Time Spent: 1h 10m  (was: 1h)

> Use queue name for address if none specified via configuration
> --------------------------------------------------------------
>                 Key: ARTEMIS-2680
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-2680
>             Project: ActiveMQ Artemis
>          Issue Type: Improvement
>            Reporter: Justin Bertram
>            Assignee: Justin Bertram
>            Priority: Minor
>          Time Spent: 1h 10m
>  Remaining Estimate: 0h
> If 
> {{org.apache.activemq.artemis.core.server.impl.ActiveMQServerImpl#createQueue()}}
>  is invoked with an effectively {{null}} address name then the queue name 
> will be used for the address. This is not true of queues created via 
> {{org.apache.activemq.artemis.core.config.CoreQueueConfiguration}}, but it 
> should be.

This message was sent by Atlassian Jira

Reply via email to