danielcweeks opened a new pull request, #12165: URL: https://github.com/apache/iceberg/pull/12165
The NestedField builder relies on the NestedField constructor to validate values, but the builder uses `Integer` for the id value whereas the constructor uses `int`. This results in a confusing auto-unboxing related NPE if a field id is not provided to the builder. This PR adds a check prior to the auto-unboxing to provide a reasonable message that the field id is required. -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
