cshannon commented on PR #965:
URL: https://github.com/apache/activemq/pull/965#issuecomment-1423292639

   I should add a quick note that I merged this but this doesn't solve all 
issues (but makes things much better). It uses the original destination and not 
the DLQ destination the message is going to which means that in some cases 
(like if destinationPerDurableSubscriber flag is true) a copy of the message 
for each destination still wouldn't be DLQ'd as more than one destination still 
shares a message audit. So while this greatly improves things it isn't perfect. 
A follow in issue/PR can be created to improve it further if desired to handle 
the more complex 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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to