xxchan commented on code in PR #1143: URL: https://github.com/apache/iceberg-rust/pull/1143#discussion_r2030393097
########## rust-toolchain.toml: ########## Review Comment: But it seems our new MSRV policy (3 month) can cover datafusion's 4 month policy, so maybe there's no need to have separate MSRV any more. However, if we only upgrade MSRV "on release", that may not be enough. (imagine datafusion releases more frequently, and we can't bump MSRV until next release.) Maybe we could just bump MSRV "on demand" in this case? We just need to guarantee at any time, the 3 month time range is covered? -- 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
