fix: resolve comments.

Author: yifanzyifanz

google-cloud-spanner/src/main/java/com/google/cloud/spanner/SpannerOptions.java

@@ -1159,11 +1159,20 @@ public Builder setEmulatorHost(String emulatorHost) {
     }
 
     /**
-     * Enable or disable leader aware routing. Leader aware routing would route all requests in
-     * RW/PDML transactions to the leader region.
+     * Enable leader aware routing. Leader aware routing would route all requests in RW/PDML
+     * transactions to the leader region.
      */
-    public Builder setLeaderAwareRouting(boolean leaderAwareRoutingEnabled) {
-      this.leaderAwareRoutingEnabled = leaderAwareRoutingEnabled;
+    public Builder enableLeaderAwareRouting() {
+      this.leaderAwareRoutingEnabled = true;
+      return this;
+    }
+
+    /**
+     * Enable leader aware routing. Leader aware routing would route all requests in RW/PDML
+     * transactions to the leader region.
+     */
+    public Builder disableLeaderAwareRouting() {
+      this.leaderAwareRoutingEnabled = false;
       return this;
     }
 

google-cloud-spanner/src/main/java/com/google/cloud/spanner/spi/v1/GapicSpannerRpc.java

@@ -1633,7 +1633,7 @@ public void deleteSession(String sessionName, @Nullable Map<Option, ?> options)
   public ApiFuture<Empty> asyncDeleteSession(String sessionName, @Nullable Map<Option, ?> options) {
     DeleteSessionRequest request = DeleteSessionRequest.newBuilder().setName(sessionName).build();
     GrpcCallContext context =
-        newCallContext(options, sessionName, request, SpannerGrpc.getDeleteSessionMethod(), false);
+        newCallContext(options, sessionName, request, SpannerGrpc.getDeleteSessionMethod());
     return spannerStub.deleteSessionCallable().futureCall(request, context);
   }
 

google-cloud-spanner/src/main/java/com/google/cloud/spanner/spi/v1/SpannerRpc.java

@@ -319,15 +319,39 @@ Session createSession(
   ApiFuture<Empty> asyncDeleteSession(String sessionName, @Nullable Map<Option, ?> options)
       throws SpannerException;
 
+  /**
+   * Performs a streaming read.
+   *
+   * @param routeToLeader Set to true to route the request to the leader region, and false to route
+   *     the request to any region. When leader aware routing is enabled, RW/PDML requests are
+   *     preferred to be routed to the leader region, and RO requests (except for
+   *     PartitionRead/PartitionQuery) are preferred to be routed to any region for optimal latency.
+   */
   StreamingCall read(
       ReadRequest request,
       ResultStreamConsumer consumer,
       @Nullable Map<Option, ?> options,
       boolean routeToLeader);
 
+  /**
+   * Executes a query.
+   *
+   * @param routeToLeader Set to true to route the request to the leader region, and false to route
+   *     the request to any region. When leader aware routing is enabled, RW/PDML requests are
+   *     preferred to be routed to the leader region, and RO requests (except for
+   *     PartitionRead/PartitionQuery) are preferred to be routed to any region for optimal latency.
+   */
   ResultSet executeQuery(
       ExecuteSqlRequest request, @Nullable Map<Option, ?> options, boolean routeToLeader);
 
+  /**
+   * Executes a query asynchronously.
+   *
+   * @param routeToLeader Set to true to route the request to the leader region, and false to route
+   *     the request to any region. When leader aware routing is enabled, RW/PDML requests are
+   *     preferred to be routed to the leader region, and RO requests (except for
+   *     PartitionRead/PartitionQuery) are preferred to be routed to any region for optimal latency.
+   */
   ApiFuture<ResultSet> executeQueryAsync(
       ExecuteSqlRequest request, @Nullable Map<Option, ?> options, boolean routeToLeader);
 
@@ -338,6 +362,14 @@ ApiFuture<ResultSet> executeQueryAsync(
   ServerStream<PartialResultSet> executeStreamingPartitionedDml(
       ExecuteSqlRequest request, @Nullable Map<Option, ?> options, Duration timeout);
 
+  /**
+   * Executes a query with streaming result.
+   *
+   * @param routeToLeader Set to true to route the request to the leader region, and false to route
+   *     the request to any region. When leader aware routing is enabled, RW/PDML requests are
+   *     preferred to be routed to the leader region, and RO requests (except for
+   *     PartitionRead/PartitionQuery) are preferred to be routed to any region for optimal latency.
+   */
   StreamingCall executeQuery(
       ExecuteSqlRequest request,
       ResultStreamConsumer consumer,
@@ -349,10 +381,26 @@ StreamingCall executeQuery(
   ApiFuture<ExecuteBatchDmlResponse> executeBatchDmlAsync(
       ExecuteBatchDmlRequest build, Map<Option, ?> options);
 
+  /**
+   * Begins a transaction.
+   *
+   * @param routeToLeader Set to true to route the request to the leader region, and false to route
+   *     the request to any region. When leader aware routing is enabled, RW/PDML requests are
+   *     preferred to be routed to the leader region, and RO requests (except for
+   *     PartitionRead/PartitionQuery) are preferred to be routed to any region for optimal latency.
+   */
   Transaction beginTransaction(
       BeginTransactionRequest request, @Nullable Map<Option, ?> options, boolean routeToLeader)
       throws SpannerException;
 
+  /**
+   * Begins a transaction asynchronously.
+   *
+   * @param routeToLeader Set to true to route the request to the leader region, and false to route
+   *     the request to any region. When leader aware routing is enabled, RW/PDML requests are
+   *     preferred to be routed to the leader region, and RO requests (except for
+   *     PartitionRead/PartitionQuery) are preferred to be routed to any region for optimal latency.
+   */
   ApiFuture<Transaction> beginTransactionAsync(
       BeginTransactionRequest request, @Nullable Map<Option, ?> options, boolean routeToLeader);