Reverted internal array comparison

Author: alex268

table/src/main/java/tech/ydb/table/values/PrimitiveValue.java

@@ -469,9 +469,9 @@ public int compareTo(Value<?> other) {
             case Double:
                 return Double.compare(getDouble(), otherValue.getDouble());
             case Bytes:
-                return Arrays.compare(getBytesUnsafe(), otherValue.getBytesUnsafe());
+                return compareArrays(getBytesUnsafe(), otherValue.getBytesUnsafe());
             case Yson:
-                return Arrays.compare(getYsonUnsafe(), otherValue.getYsonUnsafe());
+                return compareArrays(getYsonUnsafe(), otherValue.getYsonUnsafe());
             case Text:
                 return getText().compareTo(otherValue.getText());
             case Json:
@@ -517,6 +517,24 @@ private static int compareUUID(PrimitiveValue a, PrimitiveValue b) {
         return (al != bl) ? Long.compareUnsigned(al, bl) : Long.compareUnsigned(ah, bh);
     }
 
+    private static int compareArrays(byte[] a, byte[] b) {
+        if (a == b) {
+            return 0;
+        }
+
+        int i = 0;
+        int len = Math.min(a.length, b.length);
+        while (i < len && a[i] == b[i]) {
+            i++;
+        }
+
+        if (i < len) {
+            return Byte.compare(a[i], b[i]);
+        }
+
+        return a.length - b.length;
+    }
+
     // -- implementations --
 
     private static final class Bool extends PrimitiveValue {

table/src/test/java/tech/ydb/table/values/ValueComparableTest.java

@@ -88,8 +88,13 @@ public void testPrimitiveValueComparison() {
         assertLess(PrimitiveValue.newFloat(1e-3f), PrimitiveValue.newFloat(1e-2f));
         assertLess(PrimitiveValue.newDouble(1e-3d), PrimitiveValue.newDouble(1e-2d));
 
-        assertLess(PrimitiveValue.newBytes(new byte[] { 0x01 }), PrimitiveValue.newBytes(new byte[] { 0x02 }));
-        assertLess(PrimitiveValue.newYson(new byte[] { 0x01 }), PrimitiveValue.newYson(new byte[] { 0x02 }));
+        byte[] b1 = new byte[] { 0x01, 0x02 };
+        assertEquals(PrimitiveValue.newBytesOwn(b1), PrimitiveValue.newBytesOwn(b1));
+        assertEquals(PrimitiveValue.newBytes(b1), PrimitiveValue.newBytes(new byte[] { 0x01, 0x02 }));
+        assertLess(PrimitiveValue.newBytes(b1), PrimitiveValue.newBytes(new byte[] { 0x01, 0x02, 0x1 }));
+        assertLess(PrimitiveValue.newBytes(b1), PrimitiveValue.newBytes(new byte[] { 0x02 }));
+
+        assertLess(PrimitiveValue.newYson(b1), PrimitiveValue.newYson(new byte[] { 0x01, 0x03 }));
 
         assertLess(PrimitiveValue.newText("abc"), PrimitiveValue.newText("abcd"));
         assertLess(PrimitiveValue.newJson("['abc']"), PrimitiveValue.newJson("['abcd']"));