Merge pull request #716 from zalokhan/logPerformance

guarding logging from evaluation
CorfuDB · Jun 15, 2017 · 336be2f · 336be2f
2 parents e57ced7 + 6bba4f7
commit 336be2f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/infrastructure/src/main/java/org/corfudb/infrastructure/NettyServerRouter.java b/infrastructure/src/main/java/org/corfudb/infrastructure/NettyServerRouter.java
@@ -127,7 +127,9 @@ public void channelRead(ChannelHandlerContext ctx, Object msg) {
             } else {
                 if (validateEpoch(m, ctx)) {
                     // Route the message to the handler.
-                    log.trace("Message routed to {}: {}", handler.getClass().getSimpleName(), msg);
+                    if (log.isTraceEnabled()) {
+                        log.trace("Message routed to {}: {}", handler.getClass().getSimpleName(), msg);
+                    }
                     handlerWorkers.submit(() -> handler.handleMessage(m, ctx, this));
                 }
             }

diff --git a/runtime/src/main/java/org/corfudb/runtime/clients/NettyClientRouter.java b/runtime/src/main/java/org/corfudb/runtime/clients/NettyClientRouter.java
@@ -594,7 +594,9 @@ protected void channelRead0(ChannelHandlerContext ctx, CorfuMsg m) throws Except
             } else {
                 if (validateEpochAndClientID(m, ctx)) {
                     // Route the message to the handler.
-                    log.trace("Message routed to {}: {}", handler.getClass().getSimpleName(), m);
+                    if (log.isTraceEnabled()) {
+                        log.trace("Message routed to {}: {}", handler.getClass().getSimpleName(), m);
+                    }
                     handler.handleMessage(m, ctx);
                 }
             }