mattfaltyn opened a new pull request, #17302: URL: https://github.com/apache/iceberg/pull/17302
## What Check for absent Parquet column metadata before consulting nullability in `ParquetDictionaryRowGroupFilter#notNaN`. This keeps row groups readable when a float or double column was added after an older file was written. Add `notNaN` to the existing missing-column dictionary-filter regression coverage. ## Why `mayContainNulls.get(id)` was unboxed before the existing missing-column guard. For an evolved column absent from an older file, the map has no entry and reader initialization fails with a `NullPointerException`. ## Testing - `./gradlew --no-build-cache :iceberg-parquet:test --tests 'org.apache.iceberg.parquet.TestDictionaryRowGroupFilter.testColumnNotInFile' --no-daemon` - `./gradlew --no-build-cache :iceberg-parquet:test --no-daemon` - `./gradlew --no-build-cache spotlessCheck --no-daemon` - `./gradlew --no-build-cache build -x integrationTest --no-daemon` The repository-wide build passed from a byte-identical checkout without spaces in its path. Docker-backed integration tests were unavailable locally because Docker is not installed. Closes #17301 --- **AI Disclosure** - Model: GPT-5 - Platform/Tool: OpenAI Codex - Human Oversight: partially reviewed - Prompt Summary: Diagnose and fix issue #17301 with a minimal regression test, run local validation, and prepare a contribution-ready PR. -- 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]
