---

Ajay Kannan · Ajay Kannan · commit 2d43998c38d2 · 2015-10-09T09:50:17.000-07:00
yaml --- r: 989 b: refs/heads/master c: 0f5829b h: refs/heads/master i: 987: 6f0c20f v: v3
diff --git a/[refs] b/[refs]
@@ -1,4 +1,4 @@
 ---
-refs/heads/master: 7680281785c9ba3caaa8967ac2995261e53c38d4
+refs/heads/master: 0f5829b0e3eb8fb405facec6094b29d0d53e3310
 refs/heads/travis: 0fa997e2fc9c6b61b2d91e6d163655aae67d44b6
 refs/heads/gh-pages: 5a10432ecc75f29812e33a8236c900379509fe99
diff --git a/trunk/RELEASING.md b/trunk/RELEASING.md
@@ -0,0 +1,24 @@
+### Overview
+
+Most of the release process is handled by the `after_success.sh` script, triggered after Travis CI successfully completes a non-PR build.  A new artifact will be released to Maven Central Repository via Travis CI when "-SNAPSHOT" is not included in the version (as listed in the base directory's `pom.xml`).  The website and README files will also be updated automatically in this case.  When "-SNAPSHOT" is included in the version, Travis only updates the artifact in the snapshot repository.
+
+### To push a release version
+
+1. Run `utilities/update_pom_version.sh` from the repository's base directory.
+This script takes an optional argument denoting the new version.  By default, if the current version is X.Y.Z-SNAPSHOT, the script will update the version in all the pom.xml files to X.Y.Z.  If desired, another version can be supplied via command line argument instead.
+
+2. Create a PR to update the pom.xml version.
+The PR should look something like [#225](https://github.com/GoogleCloudPlatform/gcloud-java/pull/225).  After this PR is merged into GoogleCloudPlatform/gcloud-java, Travis CI will push a new website to GoogleCloudPlatform/gh-pages, push a new artifact to the Maven Central Repository, and update versions in the README files.
+
+3. Run `utilities/update_pom_version.sh` again (to include "-SNAPSHOT" in the project version).
+As mentioned before, there is an optional version argument.  By default, the script will update the version from "X.Y.Z" to "X.Y.Z+1-SNAPSHOT".  Suppose a different version is desired, for example X+1.0.0-SNAPSHOT.  Then the appropriate command to run would be `utilities/update_pom_version.sh X+1.0.0-SNAPSHOT`.
+
+4. Create and merge in another PR to reflect the updated project version.  For an example of what this PR should look like, see [#227](https://github.com/GoogleCloudPlatform/gcloud-java/pull/227).
+
+### To push a snapshot version
+
+Pushing a snapshot is completely automated.  If "-SNAPSHOT" is included in the version denoted by the base directory's pom.xml, then an updated artifact will be pushed to the snapshot repository when Travis CI successfully completes a non-PR build.
+
+### Improvements
+
+Tagging is not currently implemented, though it was discussed in #119.  If the version updates continue to be manual, a one-line git tag command can be added to `after_success.sh` to correctly tag releases.  However, if the release process becomes fully automated, tagging becomes a harder problem, as mentioned in that issue.
diff --git a/trunk/gcloud-java-core/src/main/java/com/google/gcloud/ServiceOptions.java b/trunk/gcloud-java-core/src/main/java/com/google/gcloud/ServiceOptions.java
@@ -22,7 +22,6 @@
 import static java.nio.charset.StandardCharsets.UTF_8;
 
 import com.google.api.client.extensions.appengine.http.UrlFetchTransport;
-import com.google.api.client.http.HttpRequest;
 import com.google.api.client.http.HttpRequestInitializer;
 import com.google.api.client.http.HttpTransport;
 import com.google.api.client.http.javanet.NetHttpTransport;
@@ -61,8 +60,6 @@ public abstract class ServiceOptions<
   private final AuthCredentials authCredentials;
   private final RetryParams retryParams;
   private final ServiceRpcFactory<ServiceRpcT, OptionsT> serviceRpcFactory;
-  private final int connectTimeout;
-  private final int readTimeout;
 
   public interface HttpTransportFactory extends Serializable {
     HttpTransport create();
@@ -105,8 +102,6 @@ protected abstract static class Builder<
     private AuthCredentials authCredentials;
     private RetryParams retryParams;
     private ServiceRpcFactory<ServiceRpcT, OptionsT> serviceRpcFactory;
-    private int connectTimeout = -1;
-    private int readTimeout = -1;
 
     protected Builder() {}
 
@@ -126,89 +121,35 @@ protected B self() {
       return (B) this;
     }
 
-    /**
-     * Sets project id.
-     *
-     * @return the builder.
-     */
     public B projectId(String projectId) {
       this.projectId = projectId;
       return self();
     }
 
-    /**
-     * Sets service host.
-     *
-     * @return the builder.
-     */
     public B host(String host) {
       this.host = host;
       return self();
     }
 
-    /**
-     * Sets the transport factory.
-     *
-     * @return the builder.
-     */
     public B httpTransportFactory(HttpTransportFactory httpTransportFactory) {
       this.httpTransportFactory = httpTransportFactory;
       return self();
     }
 
-    /**
-     * Sets the service authentication credentials.
-     *
-     * @return the builder.
-     */
     public B authCredentials(AuthCredentials authCredentials) {
       this.authCredentials = authCredentials;
       return self();
     }
 
-    /**
-     * Sets configuration parameters for request retries.
-     *
-     * @return the builder.
-     */
     public B retryParams(RetryParams retryParams) {
       this.retryParams = retryParams;
       return self();
     }
 
-    /**
-     * Sets the factory for rpc services.
-     *
-     * @return the builder
-     */
     public B serviceRpcFactory(ServiceRpcFactory<ServiceRpcT, OptionsT> serviceRpcFactory) {
       this.serviceRpcFactory = serviceRpcFactory;
       return self();
     }
-
-    /**
-     * Sets the timeout in milliseconds to establish a connection.
-     *
-     * @param connectTimeout connection timeout in milliseconds. 0 for an infinite timeout, a
-     * negative number for the default value (20000).
-     * @return the builder.
-     */
-    public B connectTimeout(int connectTimeout) {
-      this.connectTimeout = connectTimeout;
-      return self();
-    }
-
-    /**
-     * Sets the timeout in milliseconds to read data from an established connection.
-     *
-     * @param readTimeout read timeout in milliseconds. 0 for an infinite timeout, a
-     * negative number for the default value (20000).
-     * @return the builder.
-     */
-    public B readTimeout(int readTimeout) {
-      this.readTimeout = readTimeout;
-      return self();
-    }
   }
 
   protected ServiceOptions(Builder<ServiceRpcT, OptionsT, ?> builder) {
@@ -219,8 +160,6 @@ protected ServiceOptions(Builder<ServiceRpcT, OptionsT, ?> builder) {
     authCredentials = firstNonNull(builder.authCredentials, defaultAuthCredentials());
     retryParams = builder.retryParams;
     serviceRpcFactory = builder.serviceRpcFactory;
-    connectTimeout = builder.connectTimeout;
-    readTimeout = builder.readTimeout;
   }
 
   private static AuthCredentials defaultAuthCredentials() {
@@ -338,84 +277,33 @@ protected static String getAppEngineProjectId() {
 
   protected abstract Set<String> scopes();
 
-  /**
-   * Returns the project id. 
-   */
   public String projectId() {
     return projectId;
   }
 
-  /**
-   * Returns the service host.
-   */
   public String host() {
     return host;
   }
 
-  /**
-   * Returns the transport factory.
-   */
   public HttpTransportFactory httpTransportFactory() {
     return httpTransportFactory;
   }
 
-  /**
-   * Returns the authentication credentials.
-   */
   public AuthCredentials authCredentials() {
     return authCredentials;
   }
 
-  /**
-   * Returns configuration parameters for request retries.
-   */
   public RetryParams retryParams() {
     return retryParams != null ? retryParams : RetryParams.noRetries();
   }
 
-  /**
-   * Returns the factory for rpc services.
-   */
   public ServiceRpcFactory<ServiceRpcT, OptionsT> serviceRpcFactory() {
     return serviceRpcFactory;
   }
 
-  /**
-   * Returns a request initializer responsible for initializing requests according to service
-   * options.
-   */
   public HttpRequestInitializer httpRequestInitializer() {
     HttpTransport httpTransport = httpTransportFactory.create();
-    final HttpRequestInitializer baseRequestInitializer =
-        authCredentials().httpRequestInitializer(httpTransport, scopes());
-    return new HttpRequestInitializer() {
-      @Override
-      public void initialize(HttpRequest httpRequest) throws IOException {
-        baseRequestInitializer.initialize(httpRequest);
-        if (connectTimeout >= 0) {
-          httpRequest.setConnectTimeout(connectTimeout);
-        }
-        if (readTimeout >= 0) {
-          httpRequest.setReadTimeout(readTimeout);
-        }
-      }
-    };
-  }
-
-  /**
-   * Returns the timeout in milliseconds to establish a connection. 0 is an infinite timeout, a
-   * negative number is the default value (20000).
-   */
-  public int connectTimeout() {
-    return connectTimeout;
-  }
-
-  /**
-   * Returns the timeout in milliseconds to read from an established connection. 0 is an infinite
-   * timeout, a negative number is the default value (20000).
-   */
-  public int readTimeout() {
-    return readTimeout;
+    return authCredentials().httpRequestInitializer(httpTransport, scopes());
   }
 
   protected int baseHashCode() {
diff --git a/trunk/gcloud-java-examples/README.md b/trunk/gcloud-java-examples/README.md
@@ -20,35 +20,6 @@ Add this to your pom.xml file
 </dependency>
 ```
 
-To run examples from your command line:
-
-1. Login using gcloud SDK (`gcloud auth login` in command line)
-
-2. Set your current project using `gcloud config set project PROJECT_ID`
-
-3. Compile using Maven (`mvn compile` in command line from your base project directory)
-
-4. Run an example using Maven from command line.
-
-  Here's an example run of `DatastoreExample`.
-  
-  Note that you have to enable the Google Cloud Datastore API on the [Google Developers Console][developers-console] before running the following commands.
-  ```
-  $mvn exec:java -Dexec.mainClass="com.google.gcloud.examples.DatastoreExample" -Dexec.args="my_name add my\ comment"
-  $mvn exec:java -Dexec.mainClass="com.google.gcloud.examples.DatastoreExample" -Dexec.args="my_name display"
-  $mvn exec:java -Dexec.mainClass="com.google.gcloud.examples.DatastoreExample" -Dexec.args="my_name delete"
-  ```
-
-  Here's an example run of `StorageExample`.
-
-  Before running the example, go to the [Google Developers Console][developers-console] to ensure that Google Cloud Storage API is enabled and that you have a bucket.  Also ensure that you have a test file (`test.txt` is chosen here) to upload to Cloud Storage stored locally on your machine.
-  ```
-  $mvn exec:java -Dexec.mainClass="com.google.gcloud.examples.StorageExample" -Dexec.args="upload /path/to/test.txt <bucket_name>"
-  $mvn exec:java -Dexec.mainClass="com.google.gcloud.examples.StorageExample" -Dexec.args="list <bucket_name>"
-  $mvn exec:java -Dexec.mainClass="com.google.gcloud.examples.StorageExample" -Dexec.args="download <bucket_name> test.txt"
-  $mvn exec:java -Dexec.mainClass="com.google.gcloud.examples.StorageExample" -Dexec.args="delete <bucket_name> test.txt"
-```
-
 Contributing
 ------------
 
@@ -79,4 +50,3 @@ Apache 2.0 - See [LICENSE] for more information.
 [CONTRIBUTING]:https://github.com/GoogleCloudPlatform/gcloud-java/blob/master/CONTRIBUTING.md
 [LICENSE]: https://github.com/GoogleCloudPlatform/gcloud-java/blob/master/LICENSE
 [cloud-platform]: https://cloud.google.com/
-[developers-console]:https://console.developers.google.com/
diff --git a/trunk/gcloud-java-storage/src/main/java/com/google/gcloud/storage/Storage.java b/trunk/gcloud-java-storage/src/main/java/com/google/gcloud/storage/Storage.java
@@ -632,15 +632,9 @@ public static Builder builder() {
    * is only valid within a certain time period.
    * This is particularly useful if you don't want publicly
    * accessible blobs, but don't want to require users to explicitly log in.
-   * <p>
-   * Example usage of creating a signed URL that is valid for 2 weeks:
-   * <pre>   {@code
-   *     service.signUrl(BlobInfo.of("bucket", "name"), TimeUnit.DAYS.toSeconds(14));
-   * }</pre>
    *
    * @param blobInfo the blob associated with the signed url
    * @param  expirationTimeInSeconds the signed URL expiration (using epoch time)
-   * @param options optional URL signing options
    * @see <a href="https://cloud.google.com/storage/docs/access-control#Signed-URLs">Signed-URLs</a>
    */
   URL signUrl(BlobInfo blobInfo, long expirationTimeInSeconds, SignUrlOption... options);
diff --git a/trunk/gcloud-java-storage/src/main/java/com/google/gcloud/storage/testing/RemoteGcsHelper.java b/trunk/gcloud-java-storage/src/main/java/com/google/gcloud/storage/testing/RemoteGcsHelper.java
@@ -133,8 +133,6 @@ public static RemoteGcsHelper create(String projectId, String keyPath, Option...
               .totalRetryPeriodMillis(120000)
               .initialRetryDelayMillis(250)
               .build())
-          .connectTimeout(60000)
-          .readTimeout(60000)
           .build();
       return new RemoteGcsHelper(storageOptions);
     } catch (FileNotFoundException ex) {
