From c8c478ef81b46d41ed22ac461d005764e32aeaac Mon Sep 17 00:00:00 2001 From: Eric Anderson Date: Mon, 4 May 2015 17:07:16 -0700 Subject: [PATCH] netty: Remove goAwayStatus when client closing Client closing doesn't really many anything special, since it is still fully-operational. We don't want a later goAwayStatus() from getting squelched because we were shutting down. --- .../java/io/grpc/transport/netty/NettyClientHandler.java | 6 ------ 1 file changed, 6 deletions(-) diff --git a/netty/src/main/java/io/grpc/transport/netty/NettyClientHandler.java b/netty/src/main/java/io/grpc/transport/netty/NettyClientHandler.java index b77296e0ac..6c7f4cd55f 100644 --- a/netty/src/main/java/io/grpc/transport/netty/NettyClientHandler.java +++ b/netty/src/main/java/io/grpc/transport/netty/NettyClientHandler.java @@ -186,12 +186,6 @@ class NettyClientHandler extends Http2ConnectionHandler { stream.transportReportStatus(Status.UNKNOWN, false, new Metadata.Trailers()); } - @Override - public void close(ChannelHandlerContext ctx, ChannelPromise promise) throws Exception { - goAwayStatus(Status.UNAVAILABLE.withDescription("Network channel closed by the client")); - super.close(ctx, promise); - } - /** * Handler for the Channel shutting down. */