clebertsuconic commented on code in PR #5498: URL: https://github.com/apache/activemq-artemis/pull/5498#discussion_r1959834718
########## artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/impl/PagingStoreImpl.java: ########## @@ -1413,8 +1500,18 @@ private void installPageTransaction(final Transaction tx, final RouteContextList return; } + @Override + public boolean hasPendingIO() { + return timedWriter != null && timedWriter.hasPendingIO(); Review Comment: when timedWriter is null, we have the previous semantic. Meaning it needs to return false, as no pending writes and it can clear page status without issues. -- 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