---

mziccard · mziccard · commit f6937a949c54 · 2015-12-04T15:33:57.000+01:00
yaml --- r: 1375 b: refs/heads/master c: 338ede1 h: refs/heads/master i: 1373: f7f7ef3 1371: 31a5814 1367: ca1a863 1359: 738bb1c 1343: fe360ed
diff --git a/[refs] b/[refs]
@@ -1,5 +1,5 @@
 ---
-refs/heads/master: ae2a484006337782084ee62096a6b7114051d836
+refs/heads/master: 338ede110628da0e980bcae8c7ec499b89c12dc1
 refs/heads/travis: e21ee7b88a5edc3f3d8c71f90c3fc32abf7e8dd6
 refs/heads/gh-pages: d1b373c30c176edc08692348167bec3a244bb823
 refs/heads/bigquery: 762fa5830e6c398c0396177e3e7fd243bd62cfc3
diff --git a/trunk/gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/ExternalDataConfiguration.java b/trunk/gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/ExternalDataConfiguration.java
@@ -283,7 +283,7 @@ com.google.api.services.bigquery.model.ExternalDataConfiguration toPb() {
     if (sourceUris != null) {
       externalConfigurationPb.setSourceUris(sourceUris);
     }
-    if (formatOptions instanceof CsvOptions) {
+    if (formatOptions != null && FormatOptions.CSV.equals(formatOptions.type())) {
       externalConfigurationPb.setCsvOptions(((CsvOptions) formatOptions).toPb());
     }
     return externalConfigurationPb;
diff --git a/trunk/gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/Field.java b/trunk/gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/Field.java
@@ -182,19 +182,19 @@ public boolean equals(Object obj) {
    * than one value.
    */
   public enum Mode {
-    NULLABLE, REQUIRED, REPEATED, NOT_SET
+    NULLABLE, REQUIRED, REPEATED
   }
 
   private final String name;
   private final Type type;
-  private final Mode mode;
+  private final String mode;
   private final String description;
 
   public static final class Builder {
 
     private String name;
     private Type type;
-    private Mode mode;
+    private String mode;
     private String description;
 
     private Builder() {}
@@ -231,7 +231,7 @@ 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 = firstNonNull(mode, Mode.NOT_SET);
+      this.mode = mode != null ? mode.name() : Data.<String>nullOf(String.class);
       return this;
     }
 
@@ -279,7 +279,7 @@ public Type type() {
    * Returns the field mode. By default {@link Mode#NULLABLE} is used.
    */
   public Mode mode() {
-    return mode == Mode.NOT_SET ? null : mode;
+    return mode != null ? Mode.valueOf(mode) : null;
   }
 
   /**
@@ -329,7 +329,7 @@ TableFieldSchema toPb() {
     fieldSchemaPb.setName(name);
     fieldSchemaPb.setType(type.value().name());
     if (mode != null) {
-      fieldSchemaPb.setMode(mode == Mode.NOT_SET ? Data.<String>nullOf(String.class) : mode.name());
+      fieldSchemaPb.setMode(mode);
     }
     if (description != null) {
       fieldSchemaPb.setDescription(description);

Original file line number	Diff line number	Diff line change
`@@ -283,7 +283,7 @@ com.google.api.services.bigquery.model.ExternalDataConfiguration toPb() {`
`283`	`283`	`if (sourceUris != null) {`
`284`	`284`	`externalConfigurationPb.setSourceUris(sourceUris);`
`285`	`285`	`}`
`286`		`- if (formatOptions instanceof CsvOptions) {`
	`286`	`+ if (formatOptions != null && FormatOptions.CSV.equals(formatOptions.type())) {`
`287`	`287`	`externalConfigurationPb.setCsvOptions(((CsvOptions) formatOptions).toPb());`
`288`	`288`	`}`
`289`	`289`	`return externalConfigurationPb;`