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

    https://github.com/apache/activemq-artemis/pull/1778#discussion_r161788722
  
    --- 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 --
    
    What's the use of these setters? 
    They should not be used for serialization and I don't see why the name (or 
address) of an instantiated JMS destination would be changed.


---

Reply via email to