Remove metadata versioning

Author: KodaiD

core/src/main/java/com/scalar/db/storage/objectstorage/ObjectStorageNamespaceMetadata.java

@@ -8,30 +8,17 @@
 
 @Immutable
 public class ObjectStorageNamespaceMetadata {
-  public static final Integer DEFAULT_VERSION = 1;
   private final String name;
-  private final Integer version;
 
   @JsonCreator
-  public ObjectStorageNamespaceMetadata(
-      @JsonProperty("name") @Nullable String name,
-      @JsonProperty("version") @Nullable Integer version) {
+  public ObjectStorageNamespaceMetadata(@JsonProperty("name") @Nullable String name) {
     this.name = name != null ? name : "";
-    this.version = version != null ? version : DEFAULT_VERSION;
-  }
-
-  public ObjectStorageNamespaceMetadata(@Nullable String name) {
-    this(name, DEFAULT_VERSION);
   }
 
   public String getName() {
     return name;
   }
 
-  public Integer getVersion() {
-    return version;
-  }
-
   @Override
   public boolean equals(Object o) {
     if (this == o) {
@@ -42,11 +29,11 @@ public boolean equals(Object o) {
     }
     ObjectStorageNamespaceMetadata that = (ObjectStorageNamespaceMetadata) o;
 
-    return name.equals(that.name) && version.equals(that.version);
+    return name.equals(that.name);
   }
 
   @Override
   public int hashCode() {
-    return Objects.hash(name, version);
+    return Objects.hash(name);
   }
 }

core/src/main/java/com/scalar/db/storage/objectstorage/ObjectStorageTableMetadata.java

@@ -21,22 +21,19 @@
 @SuppressFBWarnings({"EI_EXPOSE_REP", "EI_EXPOSE_REP2"})
 @Immutable
 public class ObjectStorageTableMetadata {
-  public static final Integer DEFAULT_VERSION = 1;
   private final LinkedHashSet<String> partitionKeyNames;
   private final LinkedHashSet<String> clusteringKeyNames;
   private final Map<String, String> clusteringOrders;
   private final Set<String> secondaryIndexNames;
   private final Map<String, String> columns;
-  private final Integer version;
 
   @JsonCreator
   public ObjectStorageTableMetadata(
       @JsonProperty("partitionKeyNames") @Nullable LinkedHashSet<String> partitionKeyNames,
       @JsonProperty("clusteringKeyNames") @Nullable LinkedHashSet<String> clusteringKeyNames,
       @JsonProperty("clusteringOrders") @Nullable Map<String, String> clusteringOrders,
       @JsonProperty("secondaryIndexNames") @Nullable Set<String> secondaryIndexNames,
-      @JsonProperty("columns") @Nullable Map<String, String> columns,
-      @JsonProperty("version") @Nullable Integer version) {
+      @JsonProperty("columns") @Nullable Map<String, String> columns) {
     this.partitionKeyNames =
         partitionKeyNames != null ? new LinkedHashSet<>(partitionKeyNames) : new LinkedHashSet<>();
     this.clusteringKeyNames =
@@ -48,10 +45,9 @@ public ObjectStorageTableMetadata(
     this.secondaryIndexNames =
         secondaryIndexNames != null ? new HashSet<>(secondaryIndexNames) : Collections.emptySet();
     this.columns = columns != null ? new HashMap<>(columns) : Collections.emptyMap();
-    this.version = version != null ? version : DEFAULT_VERSION;
   }
 
-  public ObjectStorageTableMetadata(TableMetadata tableMetadata, Integer version) {
+  public ObjectStorageTableMetadata(TableMetadata tableMetadata) {
     Map<String, String> clusteringOrders =
         tableMetadata.getClusteringKeyNames().stream()
             .collect(
@@ -69,11 +65,6 @@ public ObjectStorageTableMetadata(TableMetadata tableMetadata, Integer version)
     this.clusteringOrders = clusteringOrders;
     this.secondaryIndexNames = tableMetadata.getSecondaryIndexNames();
     this.columns = columnTypeByName;
-    this.version = version;
-  }
-
-  public ObjectStorageTableMetadata(TableMetadata tableMetadata) {
-    this(tableMetadata, DEFAULT_VERSION);
   }
 
   private ObjectStorageTableMetadata(Builder builder) {
@@ -82,8 +73,7 @@ private ObjectStorageTableMetadata(Builder builder) {
         builder.clusteringKeyNames,
         builder.clusteringOrders,
         builder.secondaryIndexNames,
-        builder.columns,
-        builder.version);
+        builder.columns);
   }
 
   public LinkedHashSet<String> getPartitionKeyNames() {
@@ -106,10 +96,6 @@ public Map<String, String> getColumns() {
     return columns;
   }
 
-  public Integer getVersion() {
-    return version;
-  }
-
   @Override
   public boolean equals(Object o) {
     if (this == o) {
@@ -123,19 +109,13 @@ public boolean equals(Object o) {
         && Objects.equals(clusteringKeyNames, that.clusteringKeyNames)
         && Objects.equals(clusteringOrders, that.clusteringOrders)
         && Objects.equals(secondaryIndexNames, that.secondaryIndexNames)
-        && Objects.equals(columns, that.columns)
-        && Objects.equals(version, that.version);
+        && Objects.equals(columns, that.columns);
   }
 
   @Override
   public int hashCode() {
     return Objects.hash(
-        partitionKeyNames,
-        clusteringKeyNames,
-        clusteringOrders,
-        secondaryIndexNames,
-        columns,
-        version);
+        partitionKeyNames, clusteringKeyNames, clusteringOrders, secondaryIndexNames, columns);
   }
 
   public TableMetadata toTableMetadata() {
@@ -187,7 +167,6 @@ public static final class Builder {
     private Map<String, String> clusteringOrders;
     private Set<String> secondaryIndexNames;
     private Map<String, String> columns;
-    private Integer version;
 
     private Builder() {}
 
@@ -216,11 +195,6 @@ public ObjectStorageTableMetadata.Builder columns(Map<String, String> val) {
       return this;
     }
 
-    public ObjectStorageTableMetadata.Builder version(Integer val) {
-      version = val;
-      return this;
-    }
-
     public ObjectStorageTableMetadata build() {
       return new ObjectStorageTableMetadata(this);
     }