More detailed javadoc for options.retryParams and RetryParams

Author: mziccard

gcloud-java-core/src/main/java/com/google/gcloud/RetryParams.java

@@ -66,7 +66,6 @@ public final class RetryParams implements Serializable {
   private static final RetryParams NO_RETRIES =
       builder().retryMaxAttempts(1).retryMinAttempts(1).build();
 
-
   /**
    * RetryParams builder.
    */
@@ -206,47 +205,53 @@ public static RetryParams getDefaultInstance() {
     return DEFAULT_INSTANCE;
   }
 
+  /**
+   * Returns an instance of {@code RetryParams} where `retryMaxAttempts` and `retryMinAttempts` are
+   * set to `1`.
+   */
   public static RetryParams noRetries() {
     return NO_RETRIES;
   }
 
   /**
-   * Returns the retryMinAttempts.
+   * Returns the retryMinAttempts. Default value is {@link #DEFAULT_RETRY_MIN_ATTEMPTS}.
    */
   public int getRetryMinAttempts() {
     return retryMinAttempts;
   }
 
   /**
-   * Returns the retryMaxAttempts.
+   * Returns the retryMaxAttempts. Default value is {@link #DEFAULT_RETRY_MAX_ATTEMPTS}.
    */
   public int getRetryMaxAttempts() {
     return retryMaxAttempts;
   }
 
   /**
-   * Returns the initialRetryDelayMillis.
+   * Returns the initialRetryDelayMillis. Default value is
+   * {@link #DEFAULT_INITIAL_RETRY_DELAY_MILLIS}.
    */
   public long getInitialRetryDelayMillis() {
     return initialRetryDelayMillis;
   }
 
   /**
-   * Returns the maxRetryDelayMillis.
+   * Returns the maxRetryDelayMillis. Default values is {@link #DEFAULT_MAX_RETRY_DELAY_MILLIS}.
    */
   public long getMaxRetryDelayMillis() {
     return maxRetryDelayMillis;
   }
 
   /**
-   * Returns the maxRetryDelayBackoffFactor.
+   * Returns the maxRetryDelayBackoffFactor. Default values is
+   * {@link #DEFAULT_RETRY_DELAY_BACKOFF_FACTOR}.
    */
   public double getRetryDelayBackoffFactor() {
     return retryDelayBackoffFactor;
   }
 
   /**
-   * Returns the totalRetryPeriodMillis.
+   * Returns the totalRetryPeriodMillis. Default value is {@link #DEFAULT_TOTAL_RETRY_PERIOD_MILLIS}.
    */
   public long getTotalRetryPeriodMillis() {
     return totalRetryPeriodMillis;

gcloud-java-core/src/main/java/com/google/gcloud/ServiceOptions.java

@@ -16,7 +16,6 @@
 
 package com.google.gcloud;
 
-
 import static com.google.common.base.MoreObjects.firstNonNull;
 import static com.google.common.base.Preconditions.checkNotNull;
 import static java.nio.charset.StandardCharsets.UTF_8;
@@ -167,7 +166,8 @@ public B authCredentials(AuthCredentials authCredentials) {
     }
 
     /**
-     * Sets configuration parameters for request retries.
+     * Sets configuration parameters for request retries. If no configuration is set
+     * {@link RetryParams#noRetries()} is used.
      *
      * @return the builder.
      */
@@ -367,7 +367,8 @@ public AuthCredentials authCredentials() {
   }
 
   /**
-   * Returns configuration parameters for request retries.
+   * Returns configuration parameters for request retries. By default requests are not retried:
+   * {@link RetryParams#noRetries()} is used.
    */
   public RetryParams retryParams() {
     return retryParams != null ? retryParams : RetryParams.noRetries();