clebertsuconic commented on code in PR #4479:
URL: https://github.com/apache/activemq-artemis/pull/4479#discussion_r1195399904


##########
artemis-journal/src/main/java/org/apache/activemq/artemis/core/journal/impl/JournalImpl.java:
##########
@@ -2625,12 +2631,13 @@ private void scheduleCompact() {
          return;
       }
 
+      logger.debug("JournalImpll::scheduleCompact");

Review Comment:
   yeah.. will make it trace



##########
artemis-journal/src/main/java/org/apache/activemq/artemis/core/journal/impl/JournalImpl.java:
##########
@@ -3565,6 +3580,8 @@ protected void moveNextFile(final boolean 
scheduleReclaim, boolean blockOnClose)
 
       if (scheduleReclaim) {
          scheduleReclaim();
+      } else {
+         logger.trace("JournalImpl::moveNextFile scheduleReclaim is false ( {} 
), not calling scheduleReclaim", scheduleReclaim);

Review Comment:
   When I wrote this, I was concerned about the value changing between the 
check and the if value. so I was just reprinting just in case.. but I guess it 
can go away.



-- 
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]

Reply via email to