From da081831babdd8cd52f082de91ebdd81f4de7479 Mon Sep 17 00:00:00 2001
From: Shanika Kuruppu <skuruppu@google.com>
Date: Fri, 24 Jan 2020 14:33:53 +1100
Subject: [PATCH] Annotate SpannerRpc and Session classes as @InternalApi.

Users shouldn't be implementing these interfaces as they're internal to
the client library implementation.
---
 .../src/main/java/com/google/cloud/spanner/Session.java         | 2 ++
 .../main/java/com/google/cloud/spanner/spi/v1/SpannerRpc.java   | 2 ++
 2 files changed, 4 insertions(+)

diff --git a/google-cloud-spanner/src/main/java/com/google/cloud/spanner/Session.java b/google-cloud-spanner/src/main/java/com/google/cloud/spanner/Session.java
index 7dd7803e34..697521554e 100644
--- a/google-cloud-spanner/src/main/java/com/google/cloud/spanner/Session.java
+++ b/google-cloud-spanner/src/main/java/com/google/cloud/spanner/Session.java
@@ -17,6 +17,7 @@
 package com.google.cloud.spanner;
 
 import com.google.api.core.ApiFuture;
+import com.google.api.core.InternalApi;
 import com.google.protobuf.Empty;
 
 /**
@@ -42,6 +43,7 @@
  * require external synchronization; {@code Session} implementations are not required to be
  * thread-safe.
  */
+@InternalApi
 public interface Session extends DatabaseClient, AutoCloseable {
   /** Returns the resource name associated with this session. */
   String getName();
diff --git a/google-cloud-spanner/src/main/java/com/google/cloud/spanner/spi/v1/SpannerRpc.java b/google-cloud-spanner/src/main/java/com/google/cloud/spanner/spi/v1/SpannerRpc.java
index ec1a23a0bb..8a30cae194 100644
--- a/google-cloud-spanner/src/main/java/com/google/cloud/spanner/spi/v1/SpannerRpc.java
+++ b/google-cloud-spanner/src/main/java/com/google/cloud/spanner/spi/v1/SpannerRpc.java
@@ -17,6 +17,7 @@
 package com.google.cloud.spanner.spi.v1;
 
 import com.google.api.core.ApiFuture;
+import com.google.api.core.InternalApi;
 import com.google.api.gax.longrunning.OperationFuture;
 import com.google.cloud.ServiceRpc;
 import com.google.cloud.spanner.SpannerException;
@@ -67,6 +68,7 @@
  * is purely for expedience; a future version of this interface is likely to be independent of
  * transport to allow switching between gRPC and HTTP.
  */
+@InternalApi
 public interface SpannerRpc extends ServiceRpc {
   /** Options passed in {@link SpannerRpc} methods to control how an RPC is issued. */
   enum Option {