---

pongad · web-flow · commit de822bf64c4d · 2018-02-13T07:52:20.000+11:00
yaml --- r: 8715 b: refs/heads/master c: ce6ef37 h: refs/heads/master i: 8713: 88c4274 8711: db970b5
diff --git a/[refs] b/[refs]
@@ -1,5 +1,5 @@
 ---
-refs/heads/master: e2f0d20a82e963df7e9a788cd8c26b48b917af61
+refs/heads/master: ce6ef37c7811e96aaf97777a71e37ed321e10d1a
 refs/heads/travis: 47e4fee4fd5af9b2a8ce46f23c72ec95f9b195b2
 refs/heads/gh-pages: 6daca92127d91b7c2c99490080ecf8a13fa94cde
 refs/tags/0.0.9: 22f1839238f66c39e67ed4dfdcd273b1ae2e8444
diff --git a/trunk/google-cloud-bigquery/src/main/java/com/google/cloud/bigquery/FieldValue.java b/trunk/google-cloud-bigquery/src/main/java/com/google/cloud/bigquery/FieldValue.java
@@ -20,9 +20,9 @@
 import static com.google.common.base.Preconditions.checkState;
 
 import com.google.api.client.util.Data;
+import com.google.api.core.BetaApi;
 import com.google.common.base.MoreObjects;
 import com.google.common.io.BaseEncoding;
-
 import java.io.Serializable;
 import java.util.List;
 import java.util.Map;
@@ -65,7 +65,7 @@ public enum Attribute {
   }
 
   private FieldValue(Attribute attribute, Object value) {
-    this.attribute = attribute;
+    this.attribute = checkNotNull(attribute);
     this.value = value;
   }
 
@@ -251,7 +251,22 @@ public final boolean equals(Object obj) {
     return attribute == other.attribute && Objects.equals(value, other.value);
   }
 
-  static FieldValue of(Attribute attribute, Object value) {
+  /**
+   * Creates an instance of {@code FieldValue}, useful for testing.
+   *
+   * <p>If the {@code attribute} is {@link Attribute#PRIMITIVE}, the {@code value} should be the
+   * string representation of the underlying value, eg {@code "123"} for number {@code 123}.
+   *
+   * <p>If the {@code attribute} is {@link Attribute#REPEATED} or {@link Attribute#RECORD}, the
+   * {@code value} should be {@code List} of {@link FieldValue}s or {@link FieldValueList},
+   * respectively.
+   *
+   * <p>This method is unstable. See <a
+   * href="https://github.com/GoogleCloudPlatform/google-cloud-java/pull/2891">this discussion</a>
+   * for more context.
+   */
+  @BetaApi
+  public static FieldValue of(Attribute attribute, Object value) {
     return new FieldValue(attribute, value);
   }
 
diff --git a/trunk/google-cloud-bigquery/src/main/java/com/google/cloud/bigquery/FieldValueList.java b/trunk/google-cloud-bigquery/src/main/java/com/google/cloud/bigquery/FieldValueList.java
@@ -16,6 +16,7 @@
 
 package com.google.cloud.bigquery;
 
+import com.google.api.core.BetaApi;
 import com.google.common.collect.ImmutableList;
 import java.io.Serializable;
 import java.util.AbstractList;
@@ -84,11 +85,27 @@ public int size() {
     return row.size();
   }
 
-  static FieldValueList of(List<FieldValue> row, FieldList schema) {
+  /**
+   * Creates an instance of {@code FieldValueList}, useful for testing.
+   *
+   * <p>This method is unstable. See <a
+   * href="https://github.com/GoogleCloudPlatform/google-cloud-java/pull/2891">this discussion</a>
+   * for more context.
+   */
+  @BetaApi
+  public static FieldValueList of(List<FieldValue> row, FieldList schema) {
     return new FieldValueList(row, schema);
   }
 
-  static FieldValueList of(List<FieldValue> row, Field... schema) {
+  /**
+   * Creates an instance of {@code FieldValueList}, useful for testing.
+   *
+   * <p>This method is unstable. See <a
+   * href="https://github.com/GoogleCloudPlatform/google-cloud-java/pull/2891">this discussion</a>
+   * for more context.
+   */
+  @BetaApi
+  public static FieldValueList of(List<FieldValue> row, Field... schema) {
     return of(row, schema.length > 0 ? FieldList.of(schema) : null);
   }
 
