ZENOTME opened a new pull request, #56: URL: https://github.com/apache/iceberg-rust/pull/56
The design of Manifest List follow https://iceberg.apache.org/spec/#writer-requirements basically. - For the field in V1 is optional but require in V2 and spec didn't state the default value , we make it optional in unifed view. - But for this kind of field with indication of default value, e.g. https://iceberg.apache.org/spec/#partitioning:~:text=The%20type%20of%20files%20tracked%20by%20the%20manifest%2C%20either%20data%20or%20delete%20files%3B%200%20for%20all%20v1%20manifests, we make it require in unifed view. -- 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
