lavocatt commented on code in PR #5220: URL: https://github.com/apache/activemq-artemis/pull/5220#discussion_r1863543074
########## artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/connect/mirror/AMQPMirrorControllerSource.java: ########## @@ -578,6 +644,9 @@ public void preAcknowledge(final Transaction tx, final MessageReference ref, fin String nodeID = idSupplier.getServerID(ref); // notice the brokerID will be null for any message generated on this broker. long internalID = idSupplier.getID(ref); Message messageCommand = createMessage(ref.getQueue().getAddress(), ref.getQueue().getName(), POST_ACK, nodeID, internalID, reason); + if (noForwardSource != null) { + messageCommand.setBrokerProperty(RECEIVER_ID_FILTER, noForwardSource); + } Review Comment: Do you mean only if the decision to discard/send was not possible to take at this moment? That works for me. -- 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