Do not extend public interface that is already accessible.

services/src/main/java/io/grpc/services/BinaryLogProviderImpl.java

@@ -20,6 +20,7 @@
 import io.grpc.CallOptions;
 import io.grpc.ClientInterceptor;
 import io.grpc.ServerInterceptor;
+import io.grpc.protobuf.services.BinaryLogSink;
 import java.io.IOException;
 import java.util.concurrent.atomic.AtomicLong;
 import javax.annotation.Nullable;

services/src/main/java/io/grpc/services/BinlogHelper.java

@@ -52,6 +52,7 @@
 import io.grpc.binarylog.v1.GrpcLogEntry;
 import io.grpc.binarylog.v1.GrpcLogEntry.EventType;
 import io.grpc.binarylog.v1.Message;
+import io.grpc.protobuf.services.BinaryLogSink;
 import java.net.Inet4Address;
 import java.net.Inet6Address;
 import java.net.InetAddress;

services/src/main/java/io/grpc/services/TempFileSink.java

@@ -29,7 +29,7 @@
  * The output file goes to the JVM's temp dir with a prefix of BINARY_INFO. The proto messages
  * are written serially using {@link MessageLite#writeDelimitedTo(OutputStream)}.
  */
-class TempFileSink implements BinaryLogSink {
+class TempFileSink implements io.grpc.protobuf.services.BinaryLogSink {
   private static final Logger logger = Logger.getLogger(TempFileSink.class.getName());
 
   private final String outPath;

services/src/test/java/io/grpc/services/BinlogHelperTest.java

@@ -70,6 +70,7 @@
 import io.grpc.internal.NoopClientCall;
 import io.grpc.internal.NoopServerCall;
 import io.grpc.protobuf.StatusProto;
+import io.grpc.protobuf.services.BinaryLogSink;
 import io.grpc.services.BinlogHelper.FactoryImpl;
 import io.grpc.services.BinlogHelper.MaybeTruncated;
 import io.grpc.services.BinlogHelper.SinkWriter;