liurenjie1024 commented on PR #1345: URL: https://github.com/apache/iceberg-rust/pull/1345#issuecomment-2889759164
> One question, should we remove this function as part of the 0.5.0 release or perhaps change it to `since = "0.5.0"` > > https://github.com/apache/iceberg-rust/blame/cfe2a980b27cda744fab8e78b4550b167b4d4d45/crates/iceberg/src/io/file_io.rs#L98 > > This was introduced in #1275 It's marked as deprecated, so I think it's better to keep it for a while so that downstream users have time to migrate them? > I also have the changes for CHANGELOG.md and DEPENDENCIES.rust.tsv queued up. Should I include those in this PR or a followup PR? This seems like a small pr, so I'm fine with including them together. -- 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
