Add comparator for `UnknownLogicalType` (#3292)

This was missing, which caused exceptions. This PR adds them including a test. It takes the comparator of the underlying physical type, but it should always compare two null values: https://github.com/apache/parquet-format/blob/471d33a3c9d46e05b8a189437a9364f959f93f09/LogicalTypes.md?plain=1#L887-L892

Fokko Driesprong committed on Aug 30, 2025, 05:30 AM

Showing 2 changed files +47 additions -0 deletions

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

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