Check if logger.isDebugEnaled() to optimize loggin

voldemort · Feb 19, 2013 · 0391625 · 0391625
1 parent f50c241
commit 0391625
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/src/java/voldemort/client/protocol/admin/StreamingClient.java b/src/java/voldemort/client/protocol/admin/StreamingClient.java
@@ -550,7 +550,9 @@ public synchronized void commitToVoldemort() {
     @SuppressWarnings({ "unchecked", "rawtypes", "unused" })
     private void commitToVoldemort(List<String> storeNamesToCommit) {
 
-        logger.debug("Trying to commit to Voldemort");
+        if(logger.isDebugEnabled()) {
+            logger.debug("Trying to commit to Voldemort");
+        }
         for(Node node: nodesToStream) {
 
             for(String store: storeNamesToCommit) {
@@ -587,8 +589,10 @@ private void commitToVoldemort(List<String> storeNamesToCommit) {
                             throw new VoldemortException("Recovery Callback failed");
                         }
                     } else {
-                        logger.debug("Commit successful");
-                        logger.debug("calling checkpoint callback");
+                        if(logger.isDebugEnabled()) {
+                            logger.debug("Commit successful");
+                            logger.debug("calling checkpoint callback");
+                        }
                         Future future = streamingresults.submit(checkpointCallback);
                         try {
                             future.get();