Github user jbertram commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/1778#discussion_r161794632
  
    --- Diff: 
artemis-jms-client/src/main/java/org/apache/activemq/artemis/jms/client/ActiveMQDestination.java
 ---
    @@ -273,6 +286,14 @@ public void setAddress(String address) {
           setSimpleAddress(SimpleString.toSimpleString(address));
        }
     
    +   public void setName(String name) {
    --- End diff --
    
    The address setter was added as part of an effort to make Artemis JNDI work 
properly in Tomcat.  I assume it's needed here, but I'll remove it for 
simplicity's sake and it can be added later if necessary.


---

Reply via email to