use MinorType and check isSet

Author: tianchen92

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

@@ -665,7 +665,7 @@ public boolean equals(int index, ValueVector to, int toIndex) {
       if (to == null) {
         return false;
       }
-      if (this.getClass() != to.getClass()) {
+      if (this.getMinorType() != to.getMinorType()) {
         return false;
       }
       UnionVector that = (UnionVector) to;

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

@@ -870,12 +870,16 @@ public boolean equals(int index, ValueVector to, int toIndex) {
     if (to == null) {
       return false;
     }
-    if (this.getClass() != to.getClass()) {
+    if (this.getMinorType() != to.getMinorType()) {
       return false;
     }
 
     BaseFixedWidthVector that = (BaseFixedWidthVector) to;
 
+    if (this.isSet(index) != that.isSet(index)) {
+      return false;
+    }
+
     int leftStart = typeWidth * index;
     int leftEnd = typeWidth * (index + 1);
 

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

@@ -1369,12 +1369,16 @@ public boolean equals(int index, ValueVector to, int toIndex) {
     if (to == null) {
       return false;
     }
-    if (this.getClass() != to.getClass()) {
+    if (this.getMinorType() != to.getMinorType()) {
       return false;
     }
 
     BaseVariableWidthVector that = (BaseVariableWidthVector) to;
 
+    if (this.isSet(index) != that.isSet(index)) {
+      return false;
+    }
+
     final int leftStart = getStartOffset(index);
     final int leftEnd = getStartOffset(index + 1);
 

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

@@ -282,7 +282,7 @@ default boolean equals(ValueVector target) {
     if (this == target) {
       return true;
     }
-    if (target == null || this.getClass() != target.getClass()) {
+    if (target == null || this.getMinorType() != target.getMinorType()) {
       return false;
     }
 

java/vector/src/main/java/org/apache/arrow/vector/complex/FixedSizeListVector.java

@@ -520,7 +520,7 @@ public boolean equals(int index, ValueVector to, int toIndex) {
     if (to == null) {
       return false;
     }
-    if (this.getClass() != to.getClass()) {
+    if (this.getMinorType() != to.getMinorType()) {
       return false;
     }
 

java/vector/src/main/java/org/apache/arrow/vector/complex/ListVector.java

@@ -430,7 +430,7 @@ public boolean equals(int index, ValueVector to, int toIndex) {
     if (to == null) {
       return false;
     }
-    if (this.getClass() != to.getClass()) {
+    if (this.getMinorType() != to.getMinorType()) {
       return false;
     }
 

java/vector/src/main/java/org/apache/arrow/vector/complex/NonNullableStructVector.java

@@ -303,7 +303,7 @@ public boolean equals(int index, ValueVector to, int toIndex) {
     if (to == null) {
       return false;
     }
-    if (this.getClass() != to.getClass()) {
+    if (this.getMinorType() != to.getMinorType()) {
       return false;
     }
     NonNullableStructVector that = (NonNullableStructVector) to;