fix some comments

Author: XuQianJin-Stars

fluss-common/src/main/java/org/apache/fluss/row/BinaryArray.java

@@ -606,7 +606,7 @@ public static BinaryArray fromPrimitiveArray(double[] arr) {
     private static BinaryArray fromPrimitiveArray(
             Object arr, int offset, int length, int elementSize) {
         final long headerInBytes = calculateHeaderInBytes(length);
-        final long valueRegionInBytes = (long) elementSize * length;
+        final long valueRegionInBytes = ((long) elementSize) * length;
 
         // must align by 8 bytes
         long totalSizeInLongs = (headerInBytes + valueRegionInBytes + 7) / 8;

fluss-common/src/main/java/org/apache/fluss/row/BinaryMap.java

@@ -133,6 +133,20 @@ public BinaryMap copy(BinaryMap reuse) {
         return reuse;
     }
 
+    @Override
+    public boolean equals(Object o) {
+        if (this == o) {
+            return true;
+        }
+        if (!(o instanceof BinaryMap)) {
+            return false;
+        }
+        final BinarySection that = (BinarySection) o;
+        return sizeInBytes == that.sizeInBytes
+                && BinarySegmentUtils.equals(
+                        segments, offset, that.segments, that.offset, sizeInBytes);
+    }
+
     @Override
     public int hashCode() {
         return BinarySegmentUtils.hash(segments, offset, sizeInBytes);

fluss-common/src/main/java/org/apache/fluss/row/GenericMap.java

@@ -139,7 +139,7 @@ public static GenericMap of(Object... values) {
 
         java.util.Map<Object, Object> javaMap = new java.util.LinkedHashMap<>();
         for (int i = 0; i < values.length; i += 2) {
-            javaMap.put(values[i], (i + 1 < values.length) ? values[i + 1] : null);
+            javaMap.put(values[i], values[i + 1]);
         }
         return new GenericMap(javaMap);
     }