netty: use varargs logger in ProtocolNegotiators

grpc · Jun 27, 2019 · 9aa3981 · 9aa3981
1 parent 023b9a3
commit 9aa3981
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/netty/src/main/java/io/grpc/netty/ProtocolNegotiators.java b/netty/src/main/java/io/grpc/netty/ProtocolNegotiators.java
@@ -904,7 +904,7 @@ protected ProtocolNegotiationHandler(ChannelHandler next) {
 
     @Override
     public final void handlerAdded(ChannelHandlerContext ctx) throws Exception {
-      negotiationLogger(ctx).log(ChannelLogLevel.DEBUG, negotiatorName + " started");
+      negotiationLogger(ctx).log(ChannelLogLevel.DEBUG, "{0} started", negotiatorName);
       handlerAdded0(ctx);
     }
 
@@ -945,7 +945,7 @@ protected final void replaceProtocolNegotiationEvent(ProtocolNegotiationEvent pn
 
     protected final void fireProtocolNegotiationEvent(ChannelHandlerContext ctx) {
       checkState(pne != null, "previous protocol negotiation event hasn't triggered");
-      negotiationLogger(ctx).log(ChannelLogLevel.INFO, negotiatorName + " completed");
+      negotiationLogger(ctx).log(ChannelLogLevel.INFO, "{0} completed", negotiatorName);
       ctx.pipeline().replace(ctx.name(), /* newName= */ null, next);
       ctx.fireUserEventTriggered(pne);
     }