manuzhang opened a new pull request, #17578: URL: https://github.com/apache/iceberg/pull/17578
## Summary - validate Parquet geometry and geography logical-type parameters against the projected Iceberg type - reuse the existing Parquet-to-Iceberg primitive conversion so omitted defaults and case-insensitive CRS matching remain consistent - add focused coverage for matching parameters, geometry and geography CRS mismatches, and geography edge-algorithm mismatches ## Context Parquet geospatial values carry their CRS and edge interpolation algorithm in the logical-type annotation rather than in the WKB payload. `PruneColumns` previously ignored these parameters, so a file could be interpreted using incompatible parameters from the projected Iceberg schema. This is the Java-side counterpart discovered while reviewing [apache/iceberg-cpp#880](https://github.com/apache/iceberg-cpp/pull/880). ## Testing - `./gradlew :iceberg-parquet:test --tests org.apache.iceberg.parquet.TestPruneColumns` - `./gradlew :iceberg-parquet:spotlessJavaCheck` - `git diff --check` --- **AI Disclosure** - Model: GPT-5 - Platform/Tool: Codex - Human Oversight: User requested the fix; this draft is awaiting human review. - Prompt Summary: Compare the Iceberg C++ geospatial reader behavior with Java, confirm whether Java has the same issue, and create a Java-side fix 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]
