---

Ajay Kannan · Ajay Kannan · commit 447c4bad36d5 · 2015-10-16T14:51:37.000-07:00
yaml --- r: 3785 b: refs/heads/pubsub-alpha c: 02a56a8 h: refs/heads/master i: 3783: 5cea3d2
diff --git a/[refs] b/[refs]
@@ -2,7 +2,7 @@
 refs/heads/master: 36a62ef856d199f8efd09501b5ba65c422c01f23
 refs/heads/travis: e21ee7b88a5edc3f3d8c71f90c3fc32abf7e8dd6
 refs/heads/gh-pages: 7406918e071dd2c5677a638ae2a06e7592b6542c
-refs/heads/pubsub-alpha: af6ab8f227fb4a877973bbc03be69770bc38af2c
+refs/heads/pubsub-alpha: 02a56a831bd04734dd171a553dc569637d2059c6
 refs/heads/update-datastore: 47aae517c2cb33f1dccd909adaced73ec9d0f4df
 refs/tags/0.0.9: 22f1839238f66c39e67ed4dfdcd273b1ae2e8444
 refs/tags/v0.0.10: 207ebd2a3472fddee69fe1298eb90429e3306efd
diff --git a/branches/pubsub-alpha/gcloud-java-core/src/main/java/com/google/gcloud/ServiceOptions.java b/branches/pubsub-alpha/gcloud-java-core/src/main/java/com/google/gcloud/ServiceOptions.java
@@ -122,7 +122,8 @@ protected B self() {
     }
 
     public B projectId(String projectId) {
-      this.projectId = projectId;
+      this.projectId =
+          checkNotNull(projectId, "Project ID cannot be set to null. Leave unset for default.");
       return self();
     }
 
diff --git a/branches/pubsub-alpha/gcloud-java-datastore/src/main/java/com/google/gcloud/datastore/KeyFactory.java b/branches/pubsub-alpha/gcloud-java-datastore/src/main/java/com/google/gcloud/datastore/KeyFactory.java
@@ -29,7 +29,7 @@ public final class KeyFactory extends BaseKey.Builder<KeyFactory> {
   private final String ns;
 
   public KeyFactory(String projectId) {
-    this(projectId, null);
+    this(projectId, "");
   }
 
   public KeyFactory(String projectId, String namespace) {
diff --git a/branches/pubsub-alpha/gcloud-java-datastore/src/main/java/com/google/gcloud/datastore/Validator.java b/branches/pubsub-alpha/gcloud-java-datastore/src/main/java/com/google/gcloud/datastore/Validator.java
@@ -45,9 +45,7 @@ static String validateDatabase(String projectId) {
   }
 
   static String validateNamespace(String namespace) {
-    if (Strings.isNullOrEmpty(namespace)) {
-      return "";
-    }
+    checkArgument(namespace != null, "Namespace cannot be null. Leave unset for default.");
     checkArgument(namespace.length() <= MAX_NAMESPACE_LENGTH,
         "namespace must not contain more than 100 characters");
     checkArgument(NAMESPACE_PATTERN.matcher(namespace).matches(),

Original file line number	Diff line number	Diff line change
`@@ -122,7 +122,8 @@ protected B self() {`
`122`	`122`	`}`
`123`	`123`
`124`	`124`	`public B projectId(String projectId) {`
`125`		`- this.projectId = projectId;`
	`125`	`+ this.projectId =`
	`126`	`+ checkNotNull(projectId, "Project ID cannot be set to null. Leave unset for default.");`
`126`	`127`	`return self();`
`127`	`128`	`}`
`128`	`129`
Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ public final class KeyFactory extends BaseKey.Builder<KeyFactory> {`
`29`	`29`	`private final String ns;`
`30`	`30`
`31`	`31`	`public KeyFactory(String projectId) {`
`32`		`- this(projectId, null);`
	`32`	`+ this(projectId, "");`
`33`	`33`	`}`
`34`	`34`
`35`	`35`	`public KeyFactory(String projectId, String namespace) {`