remove CompareUtility

Author: tianchen92

java/vector/src/main/codegen/templates/UnionVector.java

@@ -18,6 +18,7 @@
 import io.netty.buffer.ArrowBuf;
 import org.apache.arrow.memory.BufferAllocator;
 import org.apache.arrow.memory.ReferenceManager;
+import org.apache.arrow.util.Preconditions;
 import org.apache.arrow.vector.ValueVector;
 import org.apache.arrow.vector.types.UnionMode;
 import org.apache.arrow.vector.compare.RangeEqualsVisitor;
@@ -37,7 +38,6 @@
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.Iterator;
-import org.apache.arrow.vector.compare.CompareUtility;
 import org.apache.arrow.vector.compare.RangeEqualsVisitor;
 import org.apache.arrow.vector.complex.impl.ComplexCopier;
 import org.apache.arrow.vector.util.CallBack;
@@ -668,20 +668,13 @@ public boolean equals(int index, ValueVector to, int toIndex) {
       if (to == null) {
         return false;
       }
-      if (!this.getField().getType().equals(to.getField().getType())) {
-        return false;
-      }
-      CompareUtility.checkIndices(this, index, to, toIndex);
+      Preconditions.checkArgument(index >= 0 && index < valueCount,
+          "index %s out of range[0, %s]:", index, valueCount - 1);
+      Preconditions.checkArgument(toIndex >= 0 && toIndex < to.getValueCount(),
+          "index %s out of range[0, %s]:", index, to.getValueCount() - 1);
 
-      UnionVector that = (UnionVector) to;
-      ValueVector leftVector = getVector(index);
-      ValueVector rightVector = that.getVector(toIndex);
-
-      if (leftVector == null) {
-        return rightVector == null;
-      } else {
-        return leftVector.equals(index, rightVector, toIndex);
-      }
+      RangeEqualsVisitor visitor = new RangeEqualsVisitor(to, index, toIndex, 1);
+      return this.accept(visitor);
     }
 
     @Override

java/vector/src/main/java/org/apache/arrow/vector/BaseFixedWidthVector.java

@@ -26,7 +26,6 @@
 import org.apache.arrow.memory.util.ArrowBufPointer;
 import org.apache.arrow.memory.util.ByteFunctionHelpers;
 import org.apache.arrow.util.Preconditions;
-import org.apache.arrow.vector.compare.CompareUtility;
 import org.apache.arrow.vector.compare.RangeEqualsVisitor;
 import org.apache.arrow.vector.ipc.message.ArrowFieldNode;
 import org.apache.arrow.vector.types.pojo.Field;
@@ -877,15 +876,13 @@ public boolean equals(int index, ValueVector to, int toIndex) {
       return false;
     }
 
-    if (!this.getField().getType().equals(to.getField().getType())) {
-      return false;
-    }
-
-    CompareUtility.checkIndices(this, index, to, toIndex);
-
-    BaseFixedWidthVector that = (BaseFixedWidthVector) to;
+    Preconditions.checkArgument(index >= 0 && index < valueCount,
+        "index %s out of range[0, %s]:", index, valueCount - 1);
+    Preconditions.checkArgument(toIndex >= 0 && toIndex < to.getValueCount(),
+        "index %s out of range[0, %s]:", index, to.getValueCount() - 1);
 
-    return CompareUtility.compare(this, index, that, toIndex);
+    RangeEqualsVisitor visitor = new RangeEqualsVisitor(to, index, toIndex, 1);
+    return this.accept(visitor);
   }
 
   @Override

java/vector/src/main/java/org/apache/arrow/vector/BaseVariableWidthVector.java

@@ -28,7 +28,6 @@
 import org.apache.arrow.memory.util.ArrowBufPointer;
 import org.apache.arrow.memory.util.ByteFunctionHelpers;
 import org.apache.arrow.util.Preconditions;
-import org.apache.arrow.vector.compare.CompareUtility;
 import org.apache.arrow.vector.compare.RangeEqualsVisitor;
 import org.apache.arrow.vector.ipc.message.ArrowFieldNode;
 import org.apache.arrow.vector.types.pojo.Field;
@@ -1371,15 +1370,13 @@ public boolean equals(int index, ValueVector to, int toIndex) {
     if (to == null) {
       return false;
     }
-    if (!this.getField().getType().equals(to.getField().getType())) {
-      return false;
-    }
-
-    CompareUtility.checkIndices(this, index, to, toIndex);
-
-    BaseVariableWidthVector that = (BaseVariableWidthVector) to;
+    Preconditions.checkArgument(index >= 0 && index < valueCount,
+        "index %s out of range[0, %s]:", index, valueCount - 1);
+    Preconditions.checkArgument(toIndex >= 0 && toIndex < to.getValueCount(),
+        "index %s out of range[0, %s]:", index, to.getValueCount() - 1);
 
-    return CompareUtility.compare(this, index, that, toIndex);
+    RangeEqualsVisitor visitor = new RangeEqualsVisitor(to, index, toIndex, 1);
+    return this.accept(visitor);
   }
 
   @Override