Github user cshannon commented on the issue:
https://github.com/apache/activemq-artemis/pull/1827
@clebertsuconic - this PR looks generally ok to me and seems like a good
idea to remove the calls to the PagedMessage to avoid unnecessary page reloads.
In regards to my inline comment, I think taking out the decrementRefCount
might be ok for non large messages but I just wanted to point out that it may
cause future issues. The only other thing was I'm not sure if it is safe to
remove the synchronized block for the postAcknowledge call inside inside
RefsOperation as I haven't looked at that code closely enough.
---
