From 9e65662d648533a388441e1a2d6d96b68358786f Mon Sep 17 00:00:00 2001
From: Ajay Kannan <ajaykannan@google.com>
Date: Fri, 18 Dec 2015 10:43:58 -0800
Subject: [PATCH 1/3] Add retryable exceptions and fix checkstyle complaints

---
 .../google/gcloud/resourcemanager/Option.java |  2 +-
 .../gcloud/resourcemanager/Project.java       |  8 ++++----
 .../resourcemanager/ResourceManager.java      | 19 +++++++++----------
 .../ResourceManagerOptions.java               |  5 +++++
 .../testing/LocalResourceManagerHelper.java   | 10 +++++-----
 .../gcloud/spi/DefaultResourceManagerRpc.java | 10 ++++++++--
 6 files changed, 32 insertions(+), 22 deletions(-)

diff --git a/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/Option.java b/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/Option.java
index 853b678a2ebe..f48c057ba049 100644
--- a/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/Option.java
+++ b/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/Option.java
@@ -25,7 +25,7 @@
 import java.util.Objects;
 
 /**
- * Base class for Resource Manager operation options
+ * Base class for Resource Manager operation options.
  */
 class Option implements Serializable {
 
diff --git a/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/Project.java b/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/Project.java
index a08d7e3714be..1b79eb973da3 100644
--- a/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/Project.java
+++ b/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/Project.java
@@ -21,7 +21,7 @@
 /**
  * A Google Cloud Resource Manager project object.
  *
- * A Project is a high-level Google Cloud Platform entity. It is a container for ACLs, APIs,
+ * <p>A Project is a high-level Google Cloud Platform entity. It is a container for ACLs, APIs,
  * AppEngine Apps, VMs, and other Google Cloud Platform resources. This class' member variables are
  * immutable.  Methods that change or update the underlying Project information return a new Project
  * instance.
@@ -77,7 +77,7 @@ public Project reload() {
   /**
    * Marks the project identified by the specified project ID for deletion.
    *
-   * This method will only affect the project if the following criteria are met:
+   * <p>This method will only affect the project if the following criteria are met:
    * <ul>
    * <li>The project does not have a billing account associated with it.
    * <li>The project has a lifecycle state of {@link ProjectInfo.State#ACTIVE}.
@@ -103,7 +103,7 @@ public void delete() {
   /**
    * Restores the project identified by the specified project ID.
    *
-   * You can only use this method for a project that has a lifecycle state of
+   * <p>You can only use this method for a project that has a lifecycle state of
    * {@link ProjectInfo.State#DELETE_REQUESTED}. After deletion starts, as indicated by a lifecycle
    * state of {@link ProjectInfo.State#DELETE_IN_PROGRESS}, the project cannot be restored. The
    * caller must have modify permissions for this project.
@@ -120,7 +120,7 @@ public void undelete() {
   /**
    * Replaces the attributes of the project.
    *
-   * The caller must have modify permissions for this project.
+   * <p>The caller must have modify permissions for this project.
    *
    * @see <a
    * href="https://cloud.google.com/resource-manager/reference/rest/v1beta1/projects/update">Cloud
diff --git a/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/ResourceManager.java b/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/ResourceManager.java
index 9ae524e28017..907f164ec00c 100644
--- a/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/ResourceManager.java
+++ b/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/ResourceManager.java
@@ -22,7 +22,7 @@
 import com.google.gcloud.Service;
 import com.google.gcloud.spi.ResourceManagerRpc;
 
-import java.util.HashSet;
+import java.util.Set;
 
 /**
  * An interface for Google Cloud Resource Manager.
@@ -31,7 +31,7 @@
  */
 public interface ResourceManager extends Service<ResourceManagerOptions> {
 
-  public static final String DEFAULT_CONTENT_TYPE = "application/octet-stream";
+  String DEFAULT_CONTENT_TYPE = "application/octet-stream";
 
   /**
    * The fields of a project.
@@ -59,7 +59,7 @@ public String selector() {
     }
 
     static String selector(ProjectField... fields) {
-      HashSet<String> fieldStrings = Sets.newHashSetWithExpectedSize(fields.length + 1);
+      Set<String> fieldStrings = Sets.newHashSetWithExpectedSize(fields.length + 1);
       fieldStrings.add(PROJECT_ID.selector());
       for (ProjectField field : fields) {
         fieldStrings.add(field.selector());
@@ -71,7 +71,7 @@ static String selector(ProjectField... fields) {
   /**
    * Class for specifying project get options.
    */
-  public class ProjectGetOption extends Option {
+  class ProjectGetOption extends Option {
 
     private static final long serialVersionUID = 270185129961146874L;
 
@@ -95,7 +95,7 @@ public static ProjectGetOption fields(ProjectField... fields) {
   /**
    * Class for specifying project list options.
    */
-  public class ProjectListOption extends Option {
+  class ProjectListOption extends Option {
 
     private static final long serialVersionUID = 7888768979702012328L;
 
@@ -227,10 +227,9 @@ public static ProjectListOption fields(ProjectField... fields) {
    * Lists the projects visible to the current user.
    *
    * <p>This method returns projects in an unspecified order. New projects do not necessarily appear
-   * at
-   * the end of the list. Use {@link ProjectListOption} to filter this list, set page size, and set
-   * page tokens. Note that pagination is currently not implemented by the Cloud Resource Manager
-   * API.
+   * at the end of the list. Use {@link ProjectListOption} to filter this list, set page size, and
+   * set page tokens. Note that pagination is currently not implemented by the Cloud Resource
+   * Manager API.
    *
    * @see <a
    * href="https://cloud.google.com/resource-manager/reference/rest/v1beta1/projects/list">Cloud
@@ -264,7 +263,7 @@ public static ProjectListOption fields(ProjectField... fields) {
    * @see <a
    * href="https://cloud.google.com/resource-manager/reference/rest/v1beta1/projects/undelete">Cloud
    * Resource Manager undelete</a>
-   * @throws ResourceManagerException
+   * @throws ResourceManagerException upon failure
    */
   void undelete(String projectId);
 }
diff --git a/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/ResourceManagerOptions.java b/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/ResourceManagerOptions.java
index a57a512d2c14..5c0c4baf1ecb 100644
--- a/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/ResourceManagerOptions.java
+++ b/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/ResourceManagerOptions.java
@@ -107,6 +107,11 @@ public boolean equals(Object obj) {
     return obj instanceof ResourceManagerOptions && baseEquals((ResourceManagerOptions) obj);
   }
 
+  @Override
+  public int hashCode() {
+    return baseHashCode();
+  }
+
   @Override
   public Builder toBuilder() {
     return new Builder(this);
diff --git a/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/testing/LocalResourceManagerHelper.java b/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/testing/LocalResourceManagerHelper.java
index 7897203de567..fb2a4f57ff41 100644
--- a/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/testing/LocalResourceManagerHelper.java
+++ b/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/testing/LocalResourceManagerHelper.java
@@ -70,7 +70,7 @@ public class LocalResourceManagerHelper {
   private static final Set<Character> PERMISSIBLE_PROJECT_NAME_PUNCTUATION =
       ImmutableSet.of('-', '\'', '"', ' ', '!');
 
-  private HttpServer server;
+  private final HttpServer server;
   private final ConcurrentHashMap<String, Project> projects = new ConcurrentHashMap<>();
   private final int port;
 
@@ -257,7 +257,7 @@ private static Map<String, Object> parseListOptions(String query) {
     return options;
   }
 
-  private static final String checkForProjectErrors(Project project) {
+  private static String checkForProjectErrors(Project project) {
     if (project.getProjectId() == null) {
       return "Project ID cannot be empty.";
     }
@@ -291,9 +291,9 @@ private static final String checkForProjectErrors(Project project) {
     return null;
   }
 
-  private static final boolean isValidIdOrLabel(String value, int minLength, int maxLength) {
+  private static boolean isValidIdOrLabel(String value, int minLength, int maxLength) {
     for (char c : value.toCharArray()) {
-      if (c != '-' && !Character.isDigit(c) && (!Character.isLowerCase(c))) {
+      if (c != '-' && !Character.isDigit(c) && !Character.isLowerCase(c)) {
         return false;
       }
     }
@@ -367,7 +367,7 @@ Response list(Map<String, Object> options) {
     }
     String[] fields = (String[]) options.get("fields");
     for (Project p : projects.values()) {
-      Boolean includeProject = includeProject(p, filters);
+      boolean includeProject = includeProject(p, filters);
       if (includeProject) {
         try {
           projectsSerialized.add(jsonFactory.toString(extractFields(p, fields)));
diff --git a/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/spi/DefaultResourceManagerRpc.java b/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/spi/DefaultResourceManagerRpc.java
index 30c9d35c7616..f3240302c54e 100644
--- a/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/spi/DefaultResourceManagerRpc.java
+++ b/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/spi/DefaultResourceManagerRpc.java
@@ -5,6 +5,7 @@
 import static com.google.gcloud.spi.ResourceManagerRpc.Option.PAGE_SIZE;
 import static com.google.gcloud.spi.ResourceManagerRpc.Option.PAGE_TOKEN;
 import static java.net.HttpURLConnection.HTTP_FORBIDDEN;
+import static java.net.HttpURLConnection.HTTP_NOT_FOUND;
 
 import com.google.api.client.googleapis.json.GoogleJsonError;
 import com.google.api.client.googleapis.json.GoogleJsonResponseException;
@@ -26,6 +27,9 @@ public class DefaultResourceManagerRpc implements ResourceManagerRpc {
 
   // see https://cloud.google.com/resource-manager/v1/errors/core_errors
   private static final Set<Integer> RETRYABLE_CODES = ImmutableSet.of(503, 500, 429);
+  private static final Set<String> RETRYABLE_REASONS = ImmutableSet.of("concurrentLimitExceeded",
+      "limitExceeded", "rateLimitExceeded", "rateLimitExceededUnreg", "servingLimitExceeded",
+      "userRateLimitExceeded", "userRateLimitExceededUnreg", "variableTermLimitExceeded");
 
   private final Cloudresourcemanager resourceManager;
 
@@ -51,7 +55,9 @@ private static ResourceManagerException translate(IOException exception) {
   }
 
   private static ResourceManagerException translate(GoogleJsonError exception) {
-    boolean retryable = RETRYABLE_CODES.contains(exception.getCode());
+    boolean retryable =
+        RETRYABLE_CODES.contains(exception.getCode()) || (!exception.getErrors().isEmpty()
+            && RETRYABLE_REASONS.contains(exception.getErrors().get(0).getReason()));
     return new ResourceManagerException(exception.getCode(), exception.getMessage(), retryable);
   }
 
@@ -82,7 +88,7 @@ public Project get(String projectId, Map<Option, ?> options) throws ResourceMana
           .execute();
     } catch (IOException ex) {
       ResourceManagerException translated = translate(ex);
-      if (translated.code() == HTTP_FORBIDDEN) {
+      if (translated.code() == HTTP_FORBIDDEN || translated.code() == HTTP_NOT_FOUND) {
         return null; // Project not found
       } else {
         throw translated;

From a3f176947ebda4bb855d7c2cf75b10c0241c564f Mon Sep 17 00:00:00 2001
From: Ajay Kannan <ajaykannan@google.com>
Date: Fri, 18 Dec 2015 13:46:10 -0800
Subject: [PATCH 2/3] minor fixes

---
 .../com/google/gcloud/resourcemanager/ResourceManager.java    | 4 ++--
 .../java/com/google/gcloud/spi/DefaultResourceManagerRpc.java | 3 ++-
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/ResourceManager.java b/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/ResourceManager.java
index 907f164ec00c..5d9840362037 100644
--- a/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/ResourceManager.java
+++ b/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/ResourceManager.java
@@ -213,8 +213,8 @@ public static ProjectListOption fields(ProjectField... fields) {
   /**
    * Retrieves the project identified by the specified project ID.
    *
-   * <p>The caller must have read permissions for this project. Returns {@code null} if project
-   * not found.
+   * <p>Returns {@code null} if the project is not found or if the user doesn't have read
+   * permissions for the project.
    *
    * @see <a
    * href="https://cloud.google.com/resource-manager/reference/rest/v1beta1/projects/get">Cloud
diff --git a/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/spi/DefaultResourceManagerRpc.java b/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/spi/DefaultResourceManagerRpc.java
index f3240302c54e..ec95207c2e7b 100644
--- a/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/spi/DefaultResourceManagerRpc.java
+++ b/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/spi/DefaultResourceManagerRpc.java
@@ -89,7 +89,8 @@ public Project get(String projectId, Map<Option, ?> options) throws ResourceMana
     } catch (IOException ex) {
       ResourceManagerException translated = translate(ex);
       if (translated.code() == HTTP_FORBIDDEN || translated.code() == HTTP_NOT_FOUND) {
-        return null; // Project not found
+        // Service can return either 403 or 404 to signify that the project doesn't exist.
+        return null;
       } else {
         throw translated;
       }

From a67db1f87cf39d862a3106b6e978fc6f9f46c452 Mon Sep 17 00:00:00 2001
From: Ajay Kannan <ajaykannan@google.com>
Date: Fri, 18 Dec 2015 16:06:20 -0800
Subject: [PATCH 3/3] more checkstyle fixes

---
 .../google/gcloud/resourcemanager/testing/package-info.java    | 2 +-
 .../gcloud/resourcemanager/LocalResourceManagerHelperTest.java | 3 +--
 .../google/gcloud/resourcemanager/ResourceManagerImplTest.java | 1 -
 3 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/testing/package-info.java b/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/testing/package-info.java
index a2c07904ddbd..7e5519f7d085 100644
--- a/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/testing/package-info.java
+++ b/gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/testing/package-info.java
@@ -18,7 +18,7 @@
  * A testing helper for Google Cloud Resource Manager.
  *
  * <p>A simple usage example:
- * <p>Before the test:
+ * Before the test:
  * <pre> {@code
  * LocalResourceManagerHelper resourceManagerHelper = LocalResourceManagerHelper.create();
  * ResourceManager resourceManager = resourceManagerHelper.options().service();
diff --git a/gcloud-java-resourcemanager/src/test/java/com/google/gcloud/resourcemanager/LocalResourceManagerHelperTest.java b/gcloud-java-resourcemanager/src/test/java/com/google/gcloud/resourcemanager/LocalResourceManagerHelperTest.java
index d51685caf6ec..7eb0156d4e56 100644
--- a/gcloud-java-resourcemanager/src/test/java/com/google/gcloud/resourcemanager/LocalResourceManagerHelperTest.java
+++ b/gcloud-java-resourcemanager/src/test/java/com/google/gcloud/resourcemanager/LocalResourceManagerHelperTest.java
@@ -57,8 +57,7 @@ private static com.google.api.services.cloudresourcemanager.model.Project copyFr
         .setProjectId(from.getProjectId())
         .setName(from.getName())
         .setLabels(from.getLabels() != null ? ImmutableMap.copyOf(from.getLabels()) : null)
-        .setProjectNumber(
-            from.getProjectNumber() != null ? from.getProjectNumber().longValue() : null)
+        .setProjectNumber(from.getProjectNumber())
         .setCreateTime(from.getCreateTime())
         .setLifecycleState(from.getLifecycleState())
         .setParent(from.getParent() != null ? from.getParent().clone() : null);
diff --git a/gcloud-java-resourcemanager/src/test/java/com/google/gcloud/resourcemanager/ResourceManagerImplTest.java b/gcloud-java-resourcemanager/src/test/java/com/google/gcloud/resourcemanager/ResourceManagerImplTest.java
index 816cd2528eb3..1210e4ec81a7 100644
--- a/gcloud-java-resourcemanager/src/test/java/com/google/gcloud/resourcemanager/ResourceManagerImplTest.java
+++ b/gcloud-java-resourcemanager/src/test/java/com/google/gcloud/resourcemanager/ResourceManagerImplTest.java
@@ -218,7 +218,6 @@ public void testListFilterOptions() {
   @Test
   public void testReplace() {
     ProjectInfo createdProject = RESOURCE_MANAGER.create(COMPLETE_PROJECT);
-    String newName = "new name";
     Map<String, String> newLabels = ImmutableMap.of("new k1", "new v1");
     ProjectInfo anotherCompleteProject = ProjectInfo.builder(COMPLETE_PROJECT.projectId())
         .labels(newLabels)