diff options
author | zpencer <spencerfang@google.com> | 2017-10-27 13:15:49 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-10-27 13:15:49 -0700 |
commit | 5856a932c33ad2f4df57b304e6b1bc88ba4b27c6 (patch) | |
tree | 64e90e19a77d2fe9e9dd15ee6cb42193c749e3d8 /netty | |
parent | d840227165ad67e8bd63c2862e710467cd14ab21 (diff) | |
download | grpc-grpc-java-5856a932c33ad2f4df57b304e6b1bc88ba4b27c6.tar.gz |
netty: separate connection log (#3591)
Move netty connection log info to a separate logger:
io.grpc.netty.NettyServerTransport.connections
Users can redirect or disable this log using the usual way:
-Djava.util.logging.config.file="logging.properties"
Diffstat (limited to 'netty')
-rw-r--r-- | netty/src/main/java/io/grpc/netty/NettyServerTransport.java | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/netty/src/main/java/io/grpc/netty/NettyServerTransport.java b/netty/src/main/java/io/grpc/netty/NettyServerTransport.java index 6f2c308d5..afb124c82 100644 --- a/netty/src/main/java/io/grpc/netty/NettyServerTransport.java +++ b/netty/src/main/java/io/grpc/netty/NettyServerTransport.java @@ -38,7 +38,11 @@ import java.util.logging.Logger; * The Netty-based server transport. */ class NettyServerTransport implements ServerTransport { + @SuppressWarnings("unused") // log is for general messages, but nothing currently uses it private static final Logger log = Logger.getLogger(NettyServerTransport.class.getName()); + // connectionLog is for connection related messages only + private static final Logger connectionLog = Logger.getLogger( + String.format("%s.connections", NettyServerTransport.class.getName())); // Some exceptions are not very useful and add too much noise to the log private static final ImmutableList<String> QUIET_ERRORS = ImmutableList.of( "Connection reset by peer", @@ -156,7 +160,7 @@ class NettyServerTransport implements ServerTransport { private void notifyTerminated(Throwable t) { if (t != null) { - log.log(getLogLevel(t), "Transport failed", t); + connectionLog.log(getLogLevel(t), "Transport failed", t); } if (!terminated) { terminated = true; |