AntonRoskvist commented on PR #5593: URL: https://github.com/apache/activemq-artemis/pull/5593#issuecomment-2876614253
@jbertram @clebertsuconic Sorry for taking so long to get back to you on this. I believe these changes should cover the cases you noted. I don't really like the change on how to handle the LVQ case, though at this moment I don't know how to handle it differently while also making use of the `QueueIterateAction` (`refRemoved` has to be called before `addTail` for LVQs to keep the correct `current` value). Regardless, this should be working as intended and without locking... if you have any suggestions on a better approach for the LVQ case just let me know. -- 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
