Remove QueryHashable (#3392)

This removes the unused `QueryHashable` interface

Author: ScottDugas

fdb-record-layer-core/src/main/java/com/apple/foundationdb/record/ObjectPlanHash.java

@@ -29,7 +29,7 @@
  * member of the model apart. Using this class we can implement a stable (withstand JVM restart and class refactorings)
  * base hash for each object that can be added to the rest of the plan.
  */
-public class ObjectPlanHash implements PlanHashable, QueryHashable {
+public class ObjectPlanHash implements PlanHashable {
     @Nonnull
     private final Object id;
     private final int hashCode;
@@ -44,11 +44,6 @@ public int planHash(@Nonnull final PlanHashable.PlanHashMode mode) {
         return hashCode;
     }
 
-    @Override
-    public int queryHash(@Nonnull final QueryHashable.QueryHashKind hashKind) {
-        return hashCode;
-    }
-
     @Override
     public int hashCode() {
         return hashCode;

fdb-record-layer-core/src/main/java/com/apple/foundationdb/record/QueryHashable.java

@@ -21,7 +21,6 @@
 package com.apple.foundationdb.record;
 
 import com.apple.foundationdb.annotation.API;
-import com.apple.foundationdb.record.util.HashUtils;
 import com.google.protobuf.Descriptors;
 
 import javax.annotation.Nonnull;
@@ -31,7 +30,7 @@
  * @param <T> the result type of the function
  */
 @API(API.Status.UNSTABLE)
-public abstract class RecordFunction<T> implements PlanHashable, QueryHashable {
+public abstract class RecordFunction<T> implements PlanHashable {
 
     @Nonnull
     private final String name;
@@ -88,18 +87,4 @@ protected int basePlanHash(@Nonnull final PlanHashMode mode, ObjectPlanHash base
                 throw new UnsupportedOperationException("Hash kind " + mode.getKind() + " is not supported");
         }
     }
-
-    /**
-     * Base implementation of {@link #queryHash}.
-     * This implementation makes each concrete subclass implement its own version of {@link #queryHash} so that they are
-     * guided to add their own class modifier (See {@link com.apple.foundationdb.record.ObjectPlanHash ObjectPlanHash}).
-     * This implementation is meant to give subclasses common functionality for their own implementation.
-     * @param hashKind the query hash kind to use
-     * @param baseHash the subclass' base hash (concrete identifier)
-     * @param hashables the rest of the subclass' hashable parameters (if any)
-     * @return the query hash value calculated
-     */
-    protected int baseQueryHash(@Nonnull final QueryHashable.QueryHashKind hashKind, ObjectPlanHash baseHash, Object... hashables) {
-        return HashUtils.queryHash(hashKind, baseHash, name, hashables);
-    }
 }

fdb-record-layer-core/src/main/java/com/apple/foundationdb/record/RecordFunction.java

@@ -118,8 +118,4 @@ public int planHash(@Nonnull final PlanHashable.PlanHashMode mode) {
         return super.basePlanHash(mode, BASE_HASH, operand, index);
     }
 
-    @Override
-    public int queryHash(@Nonnull final QueryHashKind hashKind) {
-        return super.baseQueryHash(hashKind, BASE_HASH, operand, index);
-    }
 }

fdb-record-layer-core/src/main/java/com/apple/foundationdb/record/metadata/IndexRecordFunction.java

@@ -67,8 +67,4 @@ public int planHash(@Nonnull final PlanHashable.PlanHashMode mode) {
         return super.basePlanHash(mode, BASE_HASH);
     }
 
-    @Override
-    public int queryHash(@Nonnull final QueryHashKind hashKind) {
-        return super.baseQueryHash(hashKind, BASE_HASH);
-    }
 }

fdb-record-layer-core/src/main/java/com/apple/foundationdb/record/metadata/StoreRecordFunction.java

@@ -211,8 +211,4 @@ public int planHash(@Nonnull final PlanHashable.PlanHashMode mode) {
         return super.basePlanHash(mode, BASE_HASH);
     }
 
-    @Override
-    public int queryHash(@Nonnull final QueryHashKind hashKind) {
-        return super.baseQueryHash(hashKind, BASE_HASH);
-    }
 }

fdb-record-layer-core/src/main/java/com/apple/foundationdb/record/metadata/expressions/CollateFunctionKeyExpression.java

@@ -27,7 +27,6 @@
 import com.apple.foundationdb.record.metadata.Key;
 import com.apple.foundationdb.record.provider.foundationdb.FDBRecord;
 import com.apple.foundationdb.record.query.plan.cascades.KeyExpressionVisitor;
-import com.apple.foundationdb.record.util.HashUtils;
 import com.google.common.base.Verify;
 import com.google.common.collect.ImmutableList;
 import com.google.protobuf.Descriptors;
@@ -229,11 +228,6 @@ public int planHash(@Nonnull final PlanHashMode mode) {
         }
     }
 
-    @Override
-    public int queryHash(@Nonnull final QueryHashKind hashKind) {
-        return HashUtils.queryHash(hashKind, BASE_HASH, getWholeKey(), prefixSize, dimensionsSize);
-    }
-
     @Nonnull
     public static DimensionsKeyExpression of(@Nullable final KeyExpression prefix,
                                              @Nonnull final KeyExpression dimensions) {

fdb-record-layer-core/src/main/java/com/apple/foundationdb/record/metadata/expressions/DimensionsKeyExpression.java

@@ -27,7 +27,6 @@
 import com.apple.foundationdb.record.metadata.Key;
 import com.apple.foundationdb.record.provider.foundationdb.FDBRecord;
 import com.apple.foundationdb.record.query.plan.cascades.KeyExpressionVisitor;
-import com.apple.foundationdb.record.util.HashUtils;
 import com.google.protobuf.Descriptors;
 import com.google.protobuf.Message;
 
@@ -135,8 +134,4 @@ public int planHash(@Nonnull final PlanHashMode mode) {
         }
     }
 
-    @Override
-    public int queryHash(@Nonnull final QueryHashKind hashKind) {
-        return HashUtils.queryHash(hashKind, BASE_HASH);
-    }
 }

fdb-record-layer-core/src/main/java/com/apple/foundationdb/record/metadata/expressions/EmptyKeyExpression.java

@@ -33,7 +33,6 @@
 import com.apple.foundationdb.record.query.plan.cascades.KeyExpressionVisitor;
 import com.apple.foundationdb.record.query.plan.cascades.Quantifier;
 import com.apple.foundationdb.record.query.plan.cascades.expressions.ExplodeExpression;
-import com.apple.foundationdb.record.util.HashUtils;
 import com.google.common.collect.ImmutableList;
 import com.google.protobuf.Descriptors;
 import com.google.protobuf.Message;
@@ -370,11 +369,6 @@ public int planHash(@Nonnull final PlanHashMode mode) {
         }
     }
 
-    @Override
-    public int queryHash(@Nonnull final QueryHashKind hashKind) {
-        return HashUtils.queryHash(hashKind, BASE_HASH, fieldName, fanType);
-    }
-
     @Override
     public boolean equalsAtomic(AtomKeyExpression other) {
         return equals(other);

fdb-record-layer-core/src/main/java/com/apple/foundationdb/record/metadata/expressions/FieldKeyExpression.java

@@ -33,7 +33,6 @@
 import com.apple.foundationdb.record.query.plan.cascades.KeyExpressionVisitor;
 import com.apple.foundationdb.record.query.plan.cascades.values.BuiltInFunctionCatalog;
 import com.apple.foundationdb.record.query.plan.cascades.values.Value;
-import com.apple.foundationdb.record.util.HashUtils;
 import com.apple.foundationdb.record.util.ServiceLoaderProvider;
 import com.google.protobuf.Descriptors;
 import com.google.protobuf.Message;
@@ -325,20 +324,6 @@ protected int basePlanHash(@Nonnull final PlanHashMode mode, ObjectPlanHash base
         }
     }
 
-    /**
-     * Base implementation of {@link #queryHash}.
-     * This implementation makes each concrete subclass implement its own version of {@link #queryHash} so that they are
-     * guided to add their own class modifier (See {@link com.apple.foundationdb.record.ObjectPlanHash ObjectPlanHash}).
-     * This implementation is meant to give subclasses common functionality for their own implementation.
-     * @param hashKind the query hash kind to use
-     * @param baseHash the subclass' base hash (concrete identifier)
-     * @param hashables the rest of the subclass' hashable parameters (if any)
-     * @return the query hash value calculated
-     */
-    protected int baseQueryHash(@Nonnull final QueryHashKind hashKind, ObjectPlanHash baseHash, Object... hashables) {
-        return HashUtils.queryHash(hashKind, baseHash, getName(), getArguments(), hashables);
-    }
-
     @Override
     public String toString() {
         return getName() + "(" + getArguments() + ")";