Rename TableType to TableDefinition and other minor fixes

Author: mziccard

README.md

@@ -125,7 +125,6 @@ Here is a code snippet showing a simple usage example from within Compute/App En
 must [supply credentials](#authentication) and a project ID if running this snippet elsewhere.
 
 ```java
-import com.google.gcloud.bigquery.BaseTableInfo;
 import com.google.gcloud.bigquery.BigQuery;
 import com.google.gcloud.bigquery.BigQueryOptions;
 import com.google.gcloud.bigquery.Field;
@@ -137,12 +136,12 @@ import com.google.gcloud.bigquery.TableInfo;
 
 BigQuery bigquery = BigQueryOptions.defaultInstance().service();
 TableId tableId = TableId.of("dataset", "table");
-BaseTableInfo info = bigquery.getTable(tableId);
+TableInfo info = bigquery.getTable(tableId);
 if (info == null) {
   System.out.println("Creating table " + tableId);
   Field integerField = Field.of("fieldName", Field.Type.integer());
   Schema schema = Schema.of(integerField);
-  bigquery.create(TableInfo.of(tableId, DefaultTableType.of(schema)));
+  bigquery.create(TableInfo.of(tableId, DefaultTableDefinition.of(schema)));
 } else {
   System.out.println("Loading data into table " + tableId);
   LoadJobConfiguration configuration = LoadJobConfiguration.of(tableId, "gs://bucket/path");

gcloud-java-bigquery/README.md

@@ -111,7 +111,7 @@ are created from a BigQuery SQL query. In this code snippet we show how to creat
 with only one string field. Add the following imports at the top of your file:
 
 ```java
-import com.google.gcloud.bigquery.DefaultTableType;
+import com.google.gcloud.bigquery.DefaultTableDefinition;
 import com.google.gcloud.bigquery.Field;
 import com.google.gcloud.bigquery.Schema;
 import com.google.gcloud.bigquery.TableId;
@@ -126,8 +126,8 @@ Field stringField = Field.of("StringField", Field.Type.string());
 // Table schema definition
 Schema schema = Schema.of(stringField);
 // Create a table
-DefaultTableType tableType = DefaultTableType.of(schema);
-TableInfo createdTableInfo = bigquery.create(TableInfo.of(tableId, tableType));
+DefaultTableDefinition tableDefinition = DefaultTableDefinition.of(schema);
+TableInfo createdTableInfo = bigquery.create(TableInfo.of(tableId, tableDefinition));
 ```
 
 #### Loading data into a table
@@ -208,7 +208,7 @@ display on your webpage.
 import com.google.gcloud.bigquery.BigQuery;
 import com.google.gcloud.bigquery.BigQueryOptions;
 import com.google.gcloud.bigquery.DatasetInfo;
-import com.google.gcloud.bigquery.DefaultTableType;
+import com.google.gcloud.bigquery.DefaultTableDefinition;
 import com.google.gcloud.bigquery.Field;
 import com.google.gcloud.bigquery.FieldValue;
 import com.google.gcloud.bigquery.InsertAllRequest;
@@ -241,8 +241,8 @@ public class GcloudBigQueryExample {
     // Table schema definition
     Schema schema = Schema.of(stringField);
     // Create a table
-    DefaultTableType tableType = DefaultTableType.of(schema);
-    TableInfo createdTableInfo = bigquery.create(TableInfo.of(tableId, tableType));
+    DefaultTableDefinition tableDefinition = DefaultTableDefinition.of(schema);
+    TableInfo createdTableInfo = bigquery.create(TableInfo.of(tableId, tableDefinition));
 
     // Define rows to insert
     Map<String, Object> firstRow = new HashMap<>();

…oogle/gcloud/bigquery/BaseTableType.java …gcloud/bigquery/BaseTableDefinition.javagcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/BaseTableType.java renamed to gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/BaseTableDefinition.java gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/BaseTableType.java renamed to gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/BaseTableDefinition.java

@@ -27,7 +27,7 @@
 /**
  * Base class for a Google BigQuery table type.
  */
-public abstract class BaseTableType implements Serializable{
+public abstract class BaseTableDefinition implements Serializable {
 
   private static final long serialVersionUID = -374760330662959529L;
 
@@ -36,22 +36,22 @@ public abstract class BaseTableType implements Serializable{
    */
   public enum Type {
     /**
-     * A normal BigQuery table. Instances of {@code BaseTableType} for this type are implemented by
-     * {@link DefaultTableType}.
+     * A normal BigQuery table. Instances of {@code BaseTableDefinition} for this type are
+     * implemented by {@link DefaultTableDefinition}.
      */
     TABLE,
 
     /**
-     * A virtual table defined by a SQL query. Instances of {@code BaseTableType} for this type are
-     * implemented by {@link ViewType}.
+     * A virtual table defined by a SQL query. Instances of {@code BaseTableDefinition} for this
+     * type are implemented by {@link ViewDefinition}.
      *
      * @see <a href="https://cloud.google.com/bigquery/querying-data#views">Views</a>
      */
     VIEW,
 
     /**
-     * A BigQuery table backed by external data. Instances of {@code BaseTableType} for this type
-     * are implemented by {@link ExternalTableType}.
+     * A BigQuery table backed by external data. Instances of {@code BaseTableDefinition} for this
+     * type are implemented by {@link ExternalTableDefinition}.
      *
      * @see <a href="https://cloud.google.com/bigquery/federated-data-sources">Federated Data
      *     Sources</a>
@@ -68,7 +68,7 @@ public enum Type {
    * @param <T> the table type class
    * @param <B> the table type builder
    */
-  public abstract static class Builder<T extends BaseTableType, B extends Builder<T, B>> {
+  public abstract static class Builder<T extends BaseTableDefinition, B extends Builder<T, B>> {
 
     private Type type;
     private Schema schema;
@@ -77,9 +77,9 @@ public abstract static class Builder<T extends BaseTableType, B extends Builder<
       this.type = type;
     }
 
-    Builder(BaseTableType tableType) {
-      this.type = tableType.type;
-      this.schema = tableType.schema;
+    Builder(BaseTableDefinition tableDefinition) {
+      this.type = tableDefinition.type;
+      this.schema = tableDefinition.schema;
     }
 
     Builder(Table tablePb) {
@@ -113,7 +113,7 @@ public B schema(Schema schema) {
     public abstract T build();
   }
 
-  BaseTableType(Builder builder) {
+  BaseTableDefinition(Builder builder) {
     this.type = builder.type;
     this.schema = builder.schema;
   }
@@ -152,7 +152,7 @@ final int baseHashCode() {
     return Objects.hash(type);
   }
 
-  final boolean baseEquals(BaseTableType jobConfiguration) {
+  final boolean baseEquals(BaseTableDefinition jobConfiguration) {
     return Objects.equals(toPb(), jobConfiguration.toPb());
   }
 
@@ -166,14 +166,14 @@ Table toPb() {
   }
 
   @SuppressWarnings("unchecked")
-  static <T extends BaseTableType> T fromPb(Table tablePb) {
+  static <T extends BaseTableDefinition> T fromPb(Table tablePb) {
     switch (Type.valueOf(tablePb.getType())) {
       case TABLE:
-        return (T) DefaultTableType.fromPb(tablePb);
+        return (T) DefaultTableDefinition.fromPb(tablePb);
       case VIEW:
-        return (T) ViewType.fromPb(tablePb);
+        return (T) ViewDefinition.fromPb(tablePb);
       case EXTERNAL:
-        return (T) ExternalTableType.fromPb(tablePb);
+        return (T) ExternalTableDefinition.fromPb(tablePb);
       default:
         // never reached
         throw new IllegalArgumentException("Format " + tablePb.getType() + " is not supported");

gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/BigQuery.java

@@ -276,7 +276,7 @@ private TableOption(BigQueryRpc.Option option, Object value) {
      * Returns an option to specify the table's fields to be returned by the RPC call. If this
      * option is not provided all table's fields are returned. {@code TableOption.fields} can be
      * used to specify only the fields of interest. {@link TableInfo#tableId()} and
-     * {@link TableInfo#type()} are always returned, even if not specified.
+     * {@link TableInfo#definition()} are always returned, even if not specified.
      */
     public static TableOption fields(TableField... fields) {
       return new TableOption(BigQueryRpc.Option.FIELDS, TableField.selector(fields));
@@ -563,7 +563,7 @@ TableInfo getTable(TableId tableId, TableOption... options)
   /**
    * Lists the tables in the dataset. This method returns partial information on each table
    * ({@link TableInfo#tableId()}, {@link TableInfo#friendlyName()},
-   * {@link TableInfo#id()} and {@link TableInfo#type()}). To get complete information use
+   * {@link TableInfo#id()} and {@link TableInfo#definition()}). To get complete information use
    * either {@link #getTable(TableId, TableOption...)} or
    * {@link #getTable(String, String, TableOption...)}.
    *
@@ -574,9 +574,9 @@ Page<TableInfo> listTables(String datasetId, TableListOption... options)
 
   /**
    * Lists the tables in the dataset. This method returns partial information on each table
-   * ({@link TableInfo#tableId()}, {@link TableInfo#friendlyName()},
-   * {@link TableInfo#id()} and {@link TableInfo#type()}). To get complete information use
-   * either {@link #getTable(TableId, TableOption...)} or
+   * ({@link TableInfo#tableId()}, {@link TableInfo#friendlyName()}, {@link TableInfo#id()} and
+   * {@link TableInfo#definition()}). To get complete information use either
+   * {@link #getTable(TableId, TableOption...)} or
    * {@link #getTable(String, String, TableOption...)}.
    *
    * @throws BigQueryException upon failure

gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/CsvOptions.java

@@ -144,7 +144,7 @@ private CsvOptions(Builder builder) {
    * Returns whether BigQuery should accept rows that are missing trailing optional columns. If
    * {@code true}, BigQuery treats missing trailing columns as null values. If {@code false},
    * records with missing trailing columns are treated as bad records, and if the number of bad
-   * records exceeds {@link ExternalTableType#maxBadRecords()}, an invalid error is returned
+   * records exceeds {@link ExternalTableDefinition#maxBadRecords()}, an invalid error is returned
    * in the job result.
    */
   public Boolean allowJaggedRows() {

gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/Dataset.java

@@ -225,7 +225,7 @@ public Table get(String table, BigQuery.TableOption... options) {
    * @return a {@code Table} object for the created table
    * @throws BigQueryException upon failure
    */
-  public Table create(String table, BaseTableType type, BigQuery.TableOption... options) {
+  public Table create(String table, BaseTableDefinition type, BigQuery.TableOption... options) {
     TableInfo tableInfo = TableInfo.of(TableId.of(info.datasetId().dataset(), table), type);
     return new Table(bigquery, bigquery.create(tableInfo, options));
   }

…le/gcloud/bigquery/DefaultTableType.java …oud/bigquery/DefaultTableDefinition.javagcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/DefaultTableType.java renamed to gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/DefaultTableDefinition.java gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/DefaultTableType.java renamed to gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/DefaultTableDefinition.java

@@ -33,7 +33,7 @@
  *
  * @see <a href="https://cloud.google.com/bigquery/docs/tables">Managing Tables</a>
  */
-public class DefaultTableType extends BaseTableType {
+public class DefaultTableDefinition extends BaseTableDefinition {
 
   private static final long serialVersionUID = 2113445776046717900L;
 
@@ -115,7 +115,8 @@ static StreamingBuffer fromPb(Streamingbuffer streamingBufferPb) {
     }
   }
 
-  public static final class Builder extends BaseTableType.Builder<DefaultTableType, Builder> {
+  public static final class Builder
+      extends BaseTableDefinition.Builder<DefaultTableDefinition, Builder> {
 
     private Long numBytes;
     private Long numRows;
@@ -126,12 +127,12 @@ private Builder() {
       super(Type.TABLE);
     }
 
-    private Builder(DefaultTableType tableType) {
-      super(tableType);
-      this.numBytes = tableType.numBytes;
-      this.numRows = tableType.numRows;
-      this.location = tableType.location;
-      this.streamingBuffer = tableType.streamingBuffer;
+    private Builder(DefaultTableDefinition tableDefinition) {
+      super(tableDefinition);
+      this.numBytes = tableDefinition.numBytes;
+      this.numRows = tableDefinition.numRows;
+      this.location = tableDefinition.location;
+      this.streamingBuffer = tableDefinition.streamingBuffer;
     }
 
     private Builder(Table tablePb) {
@@ -167,15 +168,15 @@ Builder streamingBuffer(StreamingBuffer streamingBuffer) {
     }
 
     /**
-     * Creates a {@code DefaultTableType} object.
+     * Creates a {@code DefaultTableDefinition} object.
      */
     @Override
-    public DefaultTableType build() {
-      return new DefaultTableType(this);
+    public DefaultTableDefinition build() {
+      return new DefaultTableDefinition(this);
     }
   }
 
-  private DefaultTableType(Builder builder) {
+  private DefaultTableDefinition(Builder builder) {
     super(builder);
     this.numBytes = builder.numBytes;
     this.numRows = builder.numRows;
@@ -228,12 +229,12 @@ public static Builder builder() {
    *
    * @param schema the schema of the table
    */
-  public static DefaultTableType of(Schema schema) {
+  public static DefaultTableDefinition of(Schema schema) {
     return builder().schema(schema).build();
   }
 
   /**
-   * Returns a builder for the {@code DefaultTableType} object.
+   * Returns a builder for the {@code DefaultTableDefinition} object.
    */
   @Override
   public Builder toBuilder() {
@@ -251,7 +252,7 @@ ToStringHelper toStringHelper() {
 
   @Override
   public boolean equals(Object obj) {
-    return obj instanceof DefaultTableType && baseEquals((DefaultTableType) obj);
+    return obj instanceof DefaultTableDefinition && baseEquals((DefaultTableDefinition) obj);
   }
 
   @Override
@@ -274,7 +275,7 @@ Table toPb() {
   }
 
   @SuppressWarnings("unchecked")
-  static DefaultTableType fromPb(Table tablePb) {
+  static DefaultTableDefinition fromPb(Table tablePb) {
     return new Builder(tablePb).build();
   }
 }