fix comments, update the assertions

Author: zhangxiaotian13

spark/v3.2/spark/src/main/java/org/apache/iceberg/spark/Spark3Util.java

@@ -19,7 +19,6 @@
 
 package org.apache.iceberg.spark;
 
-import java.nio.ByteBuffer;
 import java.util.Arrays;
 import java.util.Collections;
 import java.util.List;
@@ -597,9 +596,6 @@ private static <T> String sqlString(List<org.apache.iceberg.expressions.Literal<
     private static String sqlString(org.apache.iceberg.expressions.Literal<?> lit) {
       if (lit.value() instanceof String) {
         return "'" + lit.value() + "'";
-      } else if (lit.value() instanceof ByteBuffer) {
-        // produce a literal value that Spark can parse
-        return lit.toString();
       } else {
         return lit.value().toString();
       }

spark/v3.2/spark/src/test/java/org/apache/iceberg/spark/SparkTestBase.java

@@ -19,7 +19,6 @@
 
 package org.apache.iceberg.spark;
 
-import java.lang.reflect.Array;
 import java.util.List;
 import java.util.Map;
 import java.util.stream.Collectors;
@@ -119,8 +118,6 @@ private Object[] toJava(Row row) {
             return row.getList(pos);
           } else if (value instanceof scala.collection.Map) {
             return row.getJavaMap(pos);
-          } else if (value instanceof byte[]) {
-            return IntStream.range(0, Array.getLength(value)).mapToObj(i -> Array.get(value, i)).toArray();
           } else {
             return value;
           }
@@ -160,7 +157,11 @@ private void assertEquals(String context, Object[] expectedRow, Object[] actualR
       Object actualValue = actualRow[col];
       if (expectedValue != null && expectedValue.getClass().isArray()) {
         String newContext = String.format("%s (nested col %d)", context, col + 1);
-        assertEquals(newContext, (Object[]) expectedValue, (Object[]) actualValue);
+        if (expectedValue instanceof byte[]) {
+          Assert.assertArrayEquals(newContext, (byte[]) expectedValue, (byte[]) actualValue);
+        } else {
+          assertEquals(newContext, (Object[]) expectedValue, (Object[]) actualValue);
+        }
       } else if (expectedValue != ANY) {
         Assert.assertEquals(context + " contents should match", expectedValue, actualValue);
       }

spark/v3.2/spark/src/test/java/org/apache/iceberg/spark/sql/TestSelect.java

@@ -210,7 +210,7 @@ public void testSpecifySnapshotAndTimestamp() {
   public void testBinaryInFilter() {
     sql("CREATE TABLE %s (id bigint, binary binary) USING iceberg", binaryTableName);
     sql("INSERT INTO %s VALUES (1, X''), (2, X'1111'), (3, X'11')", binaryTableName);
-    List<Object[]> expected = ImmutableList.of(row(2L, new Byte[]{0x11, 0x11}));
+    List<Object[]> expected = ImmutableList.of(row(2L, new byte[]{0x11, 0x11}));
 
     assertEquals("Should return all expected rows", expected,
         sql("SELECT id, binary FROM %s where binary > X'11'", binaryTableName));