[GitHub] [activemq-artemis] gtully commented on pull request #4483: ARTEMIS-4284 - sync operwire remove consumer with the operation conte…

["gtully (via GitHub)" <gi...@apache.org>]

gtully commented on PR #4483:
URL: https://github.com/apache/activemq-artemis/pull/4483#issuecomment-1552976577

   @clebertsuconic @jbertram I need some more eyes on this to validate that I am using the most effective way to ensure that the storage manager has completed. Is it ever possible that the storage manager does not run?
   
   the crux of the issue is that the addSorted on the queue is called from the operation context and if we don't wait on that, we can accept a new consumer that will miss these prefetched messages, and get them out of order. That whacks the redelivery accounting and messages go to the DLQ!


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