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

Reply via email to