thswlsqls opened a new pull request, #17093: URL: https://github.com/apache/iceberg/pull/17093
Closes #17092 ## Summary - `ParquetFilters.getParquetPrimitive()` handled `Number`, `CharSequence`, and `ByteBuffer` literals but not `Boolean`, so `EQ`/`NOT_EQ` predicates on boolean columns threw `UnsupportedOperationException` while building the Parquet read builder, failing the read before any rows were returned. - Adds `Boolean` to the existing `Number` branch (direct cast, same pattern), since `FilterApi.eq(BooleanColumn, Boolean)` expects the literal as-is. - No other literal types (decimal/UUID, handled separately in open PR #16621) are touched. ## Testing done - Added `TestParquet#booleanEqualityFilterPushdown` and `TestParquet#booleanInequalityFilterPushdown`, each writing a boolean-column Parquet file and reading it back through `Parquet.read(...).filter(...)`, asserting the correct rows are returned instead of throwing. - `./gradlew :iceberg-parquet:check` — 705 tests passed (0 failures/errors), spotlessCheck/checkstyle/revapi included. - `./gradlew :iceberg-parquet:revapi` — passed (no public API change; `ParquetFilters` is package-private, `getParquetPrimitive` is private static). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
