BINARY_AS_INT_96_COMPARATOR -> BINARY_AS_INT96_TIMESTAMP_COMPARATOR

Author: rahulketch

parquet-column/src/main/java/org/apache/parquet/schema/PrimitiveComparator.java

@@ -209,7 +209,7 @@ public String toString() {
   /*
    * This comparator is for comparing two timestamps represented as int96 binary.
    */
-  static final PrimitiveComparator<Binary> BINARY_AS_INT_96_COMPARATOR = new BinaryComparator() {
+  static final PrimitiveComparator<Binary> BINARY_AS_INT96_TIMESTAMP_COMPARATOR = new BinaryComparator() {
     @Override
     int compareBinary(Binary b1, Binary b2) {
       ByteBuffer bb1 = b1.toByteBuffer();
@@ -227,7 +227,7 @@ int compareBinary(Binary b1, Binary b2) {
 
     @Override
     public String toString() {
-      return "BINARY_AS_INT_96_COMPARATOR";
+      return "BINARY_AS_INT96_TIMESTAMP_COMPARATOR";
     }
   };
 

parquet-column/src/main/java/org/apache/parquet/schema/PrimitiveType.java

@@ -362,7 +362,7 @@ public <T, E extends Exception> T convert(PrimitiveTypeNameConverter<T, E> conve
 
       @Override
       PrimitiveComparator<?> comparator(LogicalTypeAnnotation logicalType) {
-        return PrimitiveComparator.BINARY_AS_INT_96_COMPARATOR;
+        return PrimitiveComparator.BINARY_AS_INT96_TIMESTAMP_COMPARATOR;
       }
     },
     FIXED_LEN_BYTE_ARRAY("getBinary", Binary.class) {

parquet-column/src/test/java/org/apache/parquet/schema/TestPrimitiveComparator.java

@@ -19,7 +19,7 @@
 package org.apache.parquet.schema;
 
 import static org.apache.parquet.schema.PrimitiveComparator.BINARY_AS_FLOAT16_COMPARATOR;
-import static org.apache.parquet.schema.PrimitiveComparator.BINARY_AS_INT_96_COMPARATOR;
+import static org.apache.parquet.schema.PrimitiveComparator.BINARY_AS_INT96_TIMESTAMP_COMPARATOR;
 import static org.apache.parquet.schema.PrimitiveComparator.BINARY_AS_SIGNED_INTEGER_COMPARATOR;
 import static org.apache.parquet.schema.PrimitiveComparator.BOOLEAN_COMPARATOR;
 import static org.apache.parquet.schema.PrimitiveComparator.DOUBLE_COMPARATOR;
@@ -302,7 +302,7 @@ public void testInt96Comparator() {
       for (int j = 0; j < valuesInAscendingOrder.length; ++j) {
         Binary bi = valuesInAscendingOrder[i];
         Binary bj = valuesInAscendingOrder[j];
-        assertEquals(Integer.compare(i, j), BINARY_AS_INT_96_COMPARATOR.compare(bi, bj));
+        assertEquals(Integer.compare(i, j), BINARY_AS_INT96_TIMESTAMP_COMPARATOR.compare(bi, bj));
       }
     }
   }