netty: Add ProtocolNegotiator.close

This notifies the negotiator when it will no longer be used, allowing it
to clean up any resources.

2 files changed, 16 insertions, 0 deletions

diff --git a/alts/src/main/java/io/grpc/alts/internal/AltsProtocolNegotiator.java b/alts/src/main/java/io/grpc/alts/internal/AltsProtocolNegotiator.java
index cd598c86a..72cd5753b 100644
--- a/alts/src/main/java/io/grpc/alts/internal/AltsProtocolNegotiator.java
+++ b/alts/src/main/java/io/grpc/alts/internal/AltsProtocolNegotiator.java
@@ -61,6 +61,11 @@ public abstract class AltsProtocolNegotiator implements ProtocolNegotiator {
                     handshakerFactory.newHandshaker(grpcHandler.getAuthority()))),
             new TsiFrameHandler());
       }
+
+      @Override
+      public void close() {
+        // TODO(jiangtaoli2016): release resources
+      }
     };
   }
 
@@ -75,6 +80,11 @@ public abstract class AltsProtocolNegotiator implements ProtocolNegotiator {
             new TsiHandshakeHandler(new NettyTsiHandshaker(handshakerFactory.newHandshaker(null))),
             new TsiFrameHandler());
       }
+
+      @Override
+      public void close() {
+        // TODO(jiangtaoli2016): release resources
+      }
     };
   }
 
diff --git a/alts/src/main/java/io/grpc/alts/internal/GoogleDefaultProtocolNegotiator.java b/alts/src/main/java/io/grpc/alts/internal/GoogleDefaultProtocolNegotiator.java
index 6bd39ef0d..b08453452 100644
--- a/alts/src/main/java/io/grpc/alts/internal/GoogleDefaultProtocolNegotiator.java
+++ b/alts/src/main/java/io/grpc/alts/internal/GoogleDefaultProtocolNegotiator.java
@@ -49,4 +49,10 @@ public final class GoogleDefaultProtocolNegotiator implements ProtocolNegotiator
       return tlsProtocolNegotiator.newHandler(grpcHandler);
     }
   }
+
+  @Override
+  public void close() {
+    altsProtocolNegotiator.close();
+    tlsProtocolNegotiator.close();
+  }
 }