---

Ajay Kannan · Ajay Kannan · commit 2c2a77ce9b52 · 2015-11-10T15:38:36.000-08:00
yaml --- r: 1167 b: refs/heads/master c: 6c40825 h: refs/heads/master i: 1165: c375dfe 1163: 70aa88d 1159: 78789d9 1151: f6dc7d2 v: v3
diff --git a/[refs] b/[refs]
@@ -1,4 +1,4 @@
 ---
-refs/heads/master: ad12f4143bbcb5d6557055452717c7380cf54959
+refs/heads/master: 6c408256e1621c7e0f3640fb7a4ffdef139f5241
 refs/heads/travis: 0fa997e2fc9c6b61b2d91e6d163655aae67d44b6
 refs/heads/gh-pages: 5a10432ecc75f29812e33a8236c900379509fe99
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
@@ -17,7 +17,7 @@
 package com.google.gcloud;
 
 import static com.google.common.base.MoreObjects.firstNonNull;
-import static com.google.common.base.Preconditions.checkNotNull;
+import static com.google.common.base.Preconditions.checkArgument;
 import static java.nio.charset.StandardCharsets.UTF_8;
 
 import com.google.api.client.extensions.appengine.http.UrlFetchTransport;
@@ -308,7 +308,10 @@ protected ServiceOptions(Class<? extends ServiceFactory<ServiceT, OptionsT>> ser
       Builder<ServiceT, ServiceRpcT, OptionsT, ?> builder) {
     projectId = builder.projectId != null ? builder.projectId : defaultProject();
     if (projectIdRequired()) {
-      checkNotNull(projectId);
+      checkArgument(
+          projectId != null,
+          "A project ID is required for this service but could not be determined from the builder or "
+          + "the environment.  Please set a project ID using the builder.");
     }
     host = firstNonNull(builder.host, defaultHost());
     httpTransportFactory = firstNonNull(builder.httpTransportFactory,
@@ -334,7 +337,7 @@ protected ServiceOptions(Class<? extends ServiceFactory<ServiceT, OptionsT>> ser
    *
    * @return true if a project ID is required to use the service, false if not.
    */
-  public boolean projectIdRequired() {
+  protected boolean projectIdRequired() {
     return true;
   }
 
