manuzhang opened a new pull request, #16673: URL: https://github.com/apache/iceberg/pull/16673
## Summary - Carry schema field paths through `SchemaParser` when parsing nested container types - Reject required `unknown` fields before constructing `NestedField` instances - Add regression coverage for struct fields, list elements, map keys, and map values ## Root Cause `SchemaParser` parsed container child types before checking the container-specific required flags. Required `unknown` values were rejected later by `NestedField`, which produced generic messages such as `element` or `value` instead of the offending schema path. ## Validation - `./gradlew :iceberg-core:test --tests org.apache.iceberg.TestSchemaParser` - `./gradlew :iceberg-core:spotlessJavaCheck` - `git diff --check upstream/main..HEAD` Addresses https://github.com/manuzhang/iceberg/issues/191 -- 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]
