gemmellr commented on PR #5128: URL: https://github.com/apache/activemq-artemis/pull/5128#issuecomment-2291506169
LGTM but I'd like someone else to review. I'd also note the latest versions have removed the specific warning log message for pageSize > pageLimitBytes...though personally, since it does log a warn when it starts blocking paging, and that mentions the file counts, I think thats already implicit and fine as-is. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] For further information, visit: https://activemq.apache.org/contact
