You are viewing a plain text version of this content. The canonical link for it is here.
Posted to gitbox@activemq.apache.org by "cshannon (via GitHub)" <gi...@apache.org> on 2023/02/16 12:27:42 UTC

[GitHub] [activemq] cshannon opened a new pull request, #980: AMQ-9217 - Fix IndividualDeadLetter strategy rollback

cshannon opened a new pull request, #980:
URL: https://github.com/apache/activemq/pull/980

   This fixes the rollback after the latest changes by using the originalDestination property to look up the correct message audit on rollback


-- 
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-unsubscribe@activemq.apache.org

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


[GitHub] [activemq] cshannon merged pull request #980: AMQ-9217 - Fix IndividualDeadLetter strategy when retrying messages

Posted by "cshannon (via GitHub)" <gi...@apache.org>.
cshannon merged PR #980:
URL: https://github.com/apache/activemq/pull/980


-- 
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-unsubscribe@activemq.apache.org

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


[GitHub] [activemq] mattrpav commented on pull request #980: AMQ-9217 - Fix IndividualDeadLetter strategy rollback

Posted by "mattrpav (via GitHub)" <gi...@apache.org>.
mattrpav commented on PR #980:
URL: https://github.com/apache/activemq/pull/980#issuecomment-1433164893

   @cshannon thanks for the PR. I'm checking out your branch and testing a scenario


-- 
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-unsubscribe@activemq.apache.org

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


[GitHub] [activemq] mattrpav commented on pull request #980: AMQ-9217 - Fix IndividualDeadLetter strategy rollback

Posted by "mattrpav (via GitHub)" <gi...@apache.org>.
mattrpav commented on PR #980:
URL: https://github.com/apache/activemq/pull/980#issuecomment-1433219405

   1. Publish 2 messages to entry queue of composite queue w/ two target queues
   2. Rollback messages off of target queues
   3. Messages moved to DLQ
   4. Invoke 'retryMessages' -- messages are moved back to their correct target queues (not the entrypoint queue)
   5. Messages are able to be re-DLQ'd with a second pass of rollback
   


-- 
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-unsubscribe@activemq.apache.org

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


[GitHub] [activemq] cshannon commented on pull request #980: AMQ-9217 - Fix IndividualDeadLetter strategy rollback

Posted by "cshannon (via GitHub)" <gi...@apache.org>.
cshannon commented on PR #980:
URL: https://github.com/apache/activemq/pull/980#issuecomment-1433013512

   This should fix the test failure here: https://builds.apache.org/job/ActiveMQ/job/ActiveMQ/job/main/245/testReport/org.apache.activemq.jmx/DLQRetryTest/testIndividualDLQ/


-- 
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-unsubscribe@activemq.apache.org

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