[GitHub] onlyMIT edited a comment on issue #2466: ARTEMIS-2206 The MQTT consumer reconnection caused the queue to not be cle…

[GitBox <gi...@apache.org>]

onlyMIT edited a comment on issue #2466: ARTEMIS-2206 The MQTT consumer reconnection caused the queue to not be cle…
URL: https://github.com/apache/activemq-artemis/pull/2466#issuecomment-453751957
 
 
   @jbertram I found an interesting Issue; That is why my RP failures in `org.apache.activemq.artemis.tests.integration.mqtt.imported.MqttClusterRemoteSubscribeTest`.The master code in `removeSubscription` method, do not use `locateQueue ` to query queue.
   
   > ServerConsumer consumer = consumers.get(address);
         consumers.remove(address);
         if (consumer != null) {
            consumer.close(false);
            consumerQoSLevels.remove(consumer.getID());
         }
   
   Tested in the master!!
   **Issue**
   Use `MqttClusterRemoteSubscribeTest` test. The internal queue name equal address name(use addressName to save `Binding`). It is wrong.
   Use MQTTTest.testNoMessageReceivedAfterUnsubscribeMQTT test. The internal queue name equal "clientId+'.'addressName"(use "clientId'.'+addressName" to save `Binding`). It is good.
   
   I created a jira [ARTEMIS-2226](https://issues.apache.org/jira/browse/ARTEMIS-2226)
   But I haven’t found out what caused this issue. Still looking for why!!!!!!!
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services