activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From amitjoy <...@git.apache.org>
Subject [GitHub] activemq-artemis pull request #1539: MQTT work
Date Mon, 18 Sep 2017 08:09:51 GMT
Github user amitjoy commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/1539#discussion_r139357643
  
    --- Diff: artemis-protocols/artemis-mqtt-protocol/src/main/java/org/apache/activemq/artemis/core/protocol/mqtt/MQTTProtocolManager.java
---
    @@ -172,4 +177,8 @@ public void invokeIncoming(MqttMessage mqttMessage, MQTTConnection
connection) {
        public void invokeOutgoing(MqttMessage mqttMessage, MQTTConnection connection) {
           super.invokeInterceptors(this.outgoingInterceptors, mqttMessage, connection);
        }
    +
    +   public Map<String, MQTTConnection> getConnectedClients() {
    +      return connectedClients;
    --- End diff --
    
    I believe it would be good to return immutable view of the map.


---

Mime
View raw message