fix: doc errors.

Author: arpan14

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

@@ -60,16 +60,17 @@ enum CallbackResponse {
     CONTINUE,
 
     /**
-     * Tell the cursor to suspend all callbacks until application calls {@link RowCursor#resume()}.
+     * Tell the cursor to suspend all callbacks until application calls {@link
+     * ForwardingAsyncResultSet#resume()}.
      */
     PAUSE,
 
     /**
      * Tell the cursor you are done receiving results, even if there are more results sitting in the
      * buffer. Once you return DONE, you will receive no further callbacks.
      *
-     * <p>Approximately equivalent to calling {@link RowCursor#cancel()}, and then returning {@code
-     * PAUSE}, but more clear, immediate, and idiomatic.
+     * <p>Approximately equivalent to calling {@link ForwardingAsyncResultSet#cancel()}, and then
+     * returning {@code PAUSE}, but more clear, immediate, and idiomatic.
      *
      * <p>It is legal to commit a transaction that owns this read before actually returning {@code
      * DONE}.
@@ -105,11 +106,12 @@ interface ReadyCallback {
    *   <li>Callback will stop being called once any of the following occurs:
    *       <ol>
    *         <li>Callback returns {@link CallbackResponse#DONE}.
-   *         <li>{@link ResultSet#tryNext()} returns {@link CursorState#DONE}.
-   *         <li>{@link ResultSet#tryNext()} throws an exception.
+   *         <li>{@link ForwardingAsyncResultSet#tryNext()} returns {@link CursorState#DONE}.
+   *         <li>{@link ForwardingAsyncResultSet#tryNext()} throws an exception.
    *       </ol>
-   *   <li>Callback may possibly be invoked after a call to {@link ResultSet#cancel()} call, but the
-   *       subsequent call to {@link #tryNext()} will yield a SpannerException.
+   *   <li>Callback may possibly be invoked after a call to {@link
+   *       ForwardingAsyncResultSet#cancel()} call, but the subsequent call to {@link #tryNext()}
+   *       will yield a SpannerException.
    *   <li>Spurious callbacks are possible where cursors are not actually ready. Typically callback
    *       should return {@link CallbackResponse#CONTINUE} any time it sees {@link
    *       CursorState#NOT_READY}.

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

@@ -18,7 +18,7 @@
 
 import com.google.api.core.ApiFuture;
 import com.google.cloud.Timestamp;
-import com.google.cloud.spanner.AsyncTransactionManager.TransactionContextFuture;
+import com.google.cloud.spanner.Options.TransactionOption;
 import com.google.cloud.spanner.TransactionManager.TransactionState;
 import com.google.common.util.concurrent.ListenableFuture;
 import com.google.common.util.concurrent.MoreExecutors;
@@ -43,7 +43,7 @@
  * so can cause resources to be leaked and deadlocks. Easiest way to guarantee this is by calling
  * {@link #close()} in a finally block.
  *
- * @see DatabaseClient#transactionManagerAsync()
+ * @see DatabaseClient#transactionManagerAsync(TransactionOption...)
  */
 public interface AsyncTransactionManager extends AutoCloseable {
   /**
@@ -91,10 +91,10 @@ Timestamp get(long timeout, TimeUnit unit)
 
   /**
    * {@link AsyncTransactionStep} is returned by {@link
-   * TransactionContextFuture#then(AsyncTransactionFunction)} and {@link
-   * AsyncTransactionStep#then(AsyncTransactionFunction)} and allows transaction steps that should
-   * be executed serially to be chained together. Each step can contain one or more statements that
-   * may execute in parallel.
+   * TransactionContextFuture#then(AsyncTransactionFunction, Executor)} and {@link
+   * AsyncTransactionStep#then(AsyncTransactionFunction, Executor)} and allows transaction steps
+   * that should be executed serially to be chained together. Each step can contain one or more
+   * statements that may execute in parallel.
    *
    * <p>Example usage:
    *
@@ -151,16 +151,16 @@ interface AsyncTransactionFunction<I, O> {
      * @param input the result of the previous transaction step.
      * @return an {@link ApiFuture} that will return the result of this step, and that will be the
      *     input of the next transaction step. This method should never return <code>null</code>.
-     *     Instead, if the method does not have a return value, the method should return {@link
-     *     ApiFutures#immediateFuture(null)}.
+     *     Instead, if the method does not have a return value, the method should return
+     *     ApiFutures#immediateFuture(null).
      */
     ApiFuture<O> apply(TransactionContext txn, I input) throws Exception;
   }
 
   /**
    * Creates a new read write transaction. This must be called before doing any other operation and
    * can only be called once. To create a new transaction for subsequent retries, see {@link
-   * #resetForRetry()}.
+   * #resetForRetryAsync()}.
    */
   TransactionContextFuture beginAsync();
 

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

@@ -32,6 +32,7 @@
 import com.google.cloud.grpc.GcpManagedChannelOptions;
 import com.google.cloud.grpc.GrpcTransportOptions;
 import com.google.cloud.spanner.Options.QueryOption;
+import com.google.cloud.spanner.Options.UpdateOption;
 import com.google.cloud.spanner.admin.database.v1.DatabaseAdminSettings;
 import com.google.cloud.spanner.admin.database.v1.stub.DatabaseAdminStubSettings;
 import com.google.cloud.spanner.admin.instance.v1.InstanceAdminSettings;
@@ -134,10 +135,7 @@ public class SpannerOptions extends ServiceOptions<Spanner, SpannerOptions> {
   private final String compressorName;
   private final boolean leaderAwareRoutingEnabled;
 
-  /**
-   * Interface that can be used to provide {@link CallCredentials} instead of {@link Credentials} to
-   * {@link SpannerOptions}.
-   */
+  /** Interface that can be used to provide {@link CallCredentials} to {@link SpannerOptions}. */
   public interface CallCredentialsProvider {
     /** Return the {@link CallCredentials} to use for a gRPC call. */
     CallCredentials getCallCredentials();
@@ -970,7 +968,7 @@ public DatabaseAdminStubSettings.Builder getDatabaseAdminStubSettingsBuilder() {
 
     /**
      * Sets a timeout specifically for Partitioned DML statements executed through {@link
-     * DatabaseClient#executePartitionedUpdate(Statement)}. The default is 2 hours.
+     * DatabaseClient#executePartitionedUpdate(Statement, UpdateOption...)}. The default is 2 hours.
      */
     public Builder setPartitionedDmlTimeout(Duration timeout) {
       this.partitionedDmlTimeout = timeout;
@@ -1065,9 +1063,7 @@ QueryOptions getEnvironmentQueryOptions() {
 
     /**
      * Sets a {@link CallCredentialsProvider} that can deliver {@link CallCredentials} to use on a
-     * per-gRPC basis. Any credentials returned by this {@link CallCredentialsProvider} will have
-     * preference above any {@link Credentials} that may have been set on the {@link SpannerOptions}
-     * instance.
+     * per-gRPC basis.
      */
     public Builder setCallCredentialsProvider(CallCredentialsProvider callCredentialsProvider) {
       this.callCredentialsProvider = callCredentialsProvider;