---

mziccard · mziccard · commit 2de0d608343e · 2015-12-03T22:31:58.000+01:00
yaml --- r: 1857 b: refs/heads/pubsub-alpha c: f84a3b1 h: refs/heads/master i: 1855: 5228324
diff --git a/[refs] b/[refs]
@@ -3,7 +3,7 @@ refs/heads/master: 689bbb466df4b2d5d2483d6edb8ac5c7c7f7c6fa
 refs/heads/travis: e21ee7b88a5edc3f3d8c71f90c3fc32abf7e8dd6
 refs/heads/gh-pages: 4e0561bb4504bf647db669a14417b2b2c87ba45d
 refs/heads/bigquery: 762fa5830e6c398c0396177e3e7fd243bd62cfc3
-refs/heads/pubsub-alpha: 79001cf046ca1f33427231655dd457cd41015e09
+refs/heads/pubsub-alpha: f84a3b115196ca702d3c4a420870edcbb3f86596
 refs/heads/resource-manager: ebf4adc5ee835cd2086c4ac5b4e78d01a5a005a7
 refs/heads/update-datastore: 482954f2c5055231e5b3122ea91d2ba00ce8187c
 refs/tags/0.0.9: 22f1839238f66c39e67ed4dfdcd273b1ae2e8444
diff --git a/branches/pubsub-alpha/gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/Field.java b/branches/pubsub-alpha/gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/Field.java
@@ -16,9 +16,11 @@
 
 package com.google.gcloud.bigquery;
 
+import static com.google.common.base.MoreObjects.firstNonNull;
 import static com.google.common.base.Preconditions.checkArgument;
 import static com.google.common.base.Preconditions.checkNotNull;
 
+import com.google.api.client.util.Data;
 import com.google.api.services.bigquery.model.TableFieldSchema;
 import com.google.common.base.Function;
 import com.google.common.base.MoreObjects;
@@ -180,7 +182,7 @@ public boolean equals(Object obj) {
    * than one value.
    */
   public enum Mode {
-    NULLABLE, REQUIRED, REPEATED
+    NULLABLE, REQUIRED, REPEATED, NOT_SET
   }
 
   private final String name;
@@ -197,6 +199,13 @@ public static final class Builder {
 
     private Builder() {}
 
+    private Builder(Field field) {
+      this.name = field.name;
+      this.type = field.type;
+      this.mode = field.mode;
+      this.description = field.description;
+    }
+
     /**
      * Sets the field name. The name must contain only letters (a-z, A-Z), numbers (0-9), or
      * underscores (_), and must start with a letter or underscore. The maximum length is 128
@@ -222,15 +231,15 @@ public Builder type(Type type) {
      * Sets the mode of the field. When not specified {@link Mode#NULLABLE} is used.
      */
     public Builder mode(Mode mode) {
-      this.mode = mode;
+      this.mode = firstNonNull(mode, Mode.NOT_SET);
       return this;
     }
 
     /**
      * Sets the field description. The maximum length is 16K characters.
      */
     public Builder description(String description) {
-      this.description = description;
+      this.description = firstNonNull(description, Data.<String>nullOf(String.class));
       return this;
     }
 
@@ -270,14 +279,14 @@ public Type type() {
    * Returns the field mode. By default {@link Mode#NULLABLE} is used.
    */
   public Mode mode() {
-    return mode;
+    return mode == Mode.NOT_SET ? null : mode;
   }
 
   /**
    * Returns the field description.
    */
   public String description() {
-    return description;
+    return Data.isNull(description) ? null : description;
   }
 
   /**
@@ -292,11 +301,7 @@ public List<Field> fields() {
    * Returns a builder for the {@code Field} object.
    */
   public Builder toBuilder() {
-    return new Builder()
-        .name(this.name)
-        .type(this.type)
-        .mode(this.mode)
-        .description(this.description);
+    return new Builder(this);
   }
 
   @Override
@@ -324,7 +329,7 @@ TableFieldSchema toPb() {
     fieldSchemaPb.setName(name);
     fieldSchemaPb.setType(type.value().name());
     if (mode != null) {
-      fieldSchemaPb.setMode(mode.name());
+      fieldSchemaPb.setMode(mode == Mode.NOT_SET ? Data.<String>nullOf(String.class) : mode.name());
     }
     if (description != null) {
       fieldSchemaPb.setDescription(description);
