ZENOTME commented on PR #349: URL: https://github.com/apache/iceberg-rust/pull/349#issuecomment-2376706216
> We're introducing a lot of new concepts here and generate a lot of open end (snapshot summary generation, metrics collection, schema compatibility checks, etc). I think it would be best to break this PR into smaller pieces. For example, I'm not sure if the way we create the fast-append is very extensible and I think we can copy a lot from PyIceberg where we track the changes of the metadata. Sorry for being late. Recently I'm back to active for this PR now. I have separated some code from this PR; for now, this mainly contains the support for the append action of the transaction. I have refactored the design of fast append to the following design of PyIceberg so that is more extensible. Feel free to make any suggestions that things need to change. cc @Fokko @liurenjie1024 @Xuanwo -- 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
