clebertsuconic commented on PR #4281: URL: https://github.com/apache/activemq-artemis/pull/4281#issuecomment-1308820712
@AntonRoskvist it is about the two consecutive rollbacks. One will add a bunch of data. The second will call the addSorted for a huge list, scanning all the lists for every rollback item. If you actually rolled back the consumer's pending first, the adds would be in order as the addSorted checks for first and last. I'm adding an optimization to keep the last added position and also check against the last added, and I will as well have the adds done outside of the locking. -- 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]
