gemmellr commented on code in PR #5173:
URL: https://github.com/apache/activemq-artemis/pull/5173#discussion_r1735021832


##########
artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/proton/AMQPConnectionContext.java:
##########
@@ -198,6 +203,18 @@ public AMQPConnectionContext(ProtonProtocolManager 
protocolManager,
       }
    }
 
+   public List<AMQPMirrorControllerTarget> getMirrorControllerTargets() {
+      return mirrorControllerTargets;
+   }
+
+   public AMQPConnectionContext 
addMirrorControllerTarget(AMQPMirrorControllerTarget mirrorControllerTarget) {
+      if (mirrorControllerTargets == null) {
+         mirrorControllerTargets = new ArrayList<>();
+      }
+      mirrorControllerTargets.add(mirrorControllerTarget);

Review Comment:
   Assuming the connection does close. This is for a specific sender that could 
itself go away for other reasons. There was just another PR yesterday for 
handling that it _didnt_ recreate the sender or close the connection in some 
such cases.



-- 
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.

To unsubscribe, e-mail: gitbox-unsubscr...@activemq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@activemq.apache.org
For additional commands, e-mail: gitbox-h...@activemq.apache.org
For further information, visit: https://activemq.apache.org/contact


Reply via email to