Fix case when ZeroVector is compared against other vector types

Author: BryanCutler

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

@@ -268,6 +268,6 @@ public void copyFromSafe(int fromIndex, int thisIndex, ValueVector from) {
 
   @Override
   public boolean accept(RangeEqualsVisitor visitor) {
-    return true;
+    return visitor.visit(this);
   }
 }

java/vector/src/main/java/org/apache/arrow/vector/compare/RangeEqualsVisitor.java

@@ -76,7 +76,7 @@ public boolean visit(UnionVector left) {
   }
 
   public boolean visit(ZeroVector left) {
-    return true;
+    return compareValueVector(left, right);
   }
 
   public boolean visit(ValueVector left) {

java/vector/src/test/java/org/apache/arrow/vector/TestValueVector.java

@@ -2265,6 +2265,19 @@ public void testZeroVectorEquals() {
     }
   }
 
+  @Test
+  public void testZeroVectorNotEquals() {
+    try (final IntVector intVector = new IntVector("int", allocator);
+         final ZeroVector zeroVector = new ZeroVector()) {
+
+      VectorEqualsVisitor zeroVisitor = new VectorEqualsVisitor(zeroVector);
+      assertFalse(intVector.accept(zeroVisitor));
+
+      VectorEqualsVisitor intVisitor = new VectorEqualsVisitor(intVector);
+      assertFalse(zeroVector.accept(intVisitor));
+    }
+  }
+
   @Test
   public void testIntVectorEqualsWithNull() {
     try (final IntVector vector1 = new IntVector("int", allocator);