liurenjie1024 commented on code in PR #1276: URL: https://github.com/apache/iceberg-rust/pull/1276#discussion_r2068298930
########## crates/iceberg/src/spec/manifest/data_file.rs: ########## @@ -151,6 +151,26 @@ pub struct DataFile { /// This field is not included in spec. It is just store in memory representation used /// in process. pub(crate) partition_spec_id: i32, + /// field id: 143 Review Comment: Since this pr is used to sync data files with v3 spec, we should also add `first_row_id` field. -- 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