dentiny opened a new pull request, #1276: URL: https://github.com/apache/iceberg-rust/pull/1276
## Which issue does this PR close? This PR sync up with upstream iceberg spec on deletion vector related fields. Reference PR: https://github.com/apache/iceberg/pull/11240 Reference file: https://github.com/apache/iceberg/blob/main/format/spec.md#manifests Context: I'm working on iceberg related projects in my company, one of the features I'm working on is deletion vector. Because there's no existing rust support for it, I need to do some of the work my own (i.e. deletion vector read and write). My philosophy is to reduce discrepancy with upstream is: - Avoid touching existing code unless unavoidable, and collaborate with open community if possible; - For deletion vector issue, I could make no other changes and keep puffin write/read completely change on my end; - The schema change in this PR is something hard to work-around. :( I personally think the schema change alone (without actual deletion vector feature implementation) make sense, because it's a sync-up from iceberg spec sync (which has already been acknowledged by the community), as the [PR](https://github.com/apache/iceberg/pull/11240). ## Are these changes tested? This PR is a no-op change, I confirm I could build + link with no problem. -- 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
