KlausDornsbach commented on code in PR #8714:
URL: https://github.com/apache/cloudstack/pull/8714#discussion_r1507522896


##########
agent/src/main/java/com/cloud/agent/Agent.java:
##########
@@ -396,17 +397,16 @@ private synchronized void scheduleHostLBCheckerTask(final 
long checkInterval) {
             hostLBTimer.cancel();
         }
         if (checkInterval > 0L) {
-            logger.info("Scheduling preferred host timer task with 
host.lb.interval=" + checkInterval + "ms");
+            logger.info("Scheduling preferred host timer task with 
host.lb.interval={}ms", checkInterval);
             hostLBTimer = new Timer("Host LB Timer");
             hostLBTimer.scheduleAtFixedRate(new PreferredHostCheckerTask(), 
checkInterval, checkInterval);
         }
     }
 
     public void scheduleWatch(final Link link, final Request request, final 
long delay, final long period) {
         synchronized (_watchList) {
-            if (logger.isDebugEnabled()) {
-                logger.debug("Adding a watch list");
-            }
+            logger.debug("Adding a watch list");

Review Comment:
   logger.debug("Adding task with request: {} to watch list", 
request.toString());
   something like this?



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