jdockerty commented on PR #1436: URL: https://github.com/apache/iceberg-rust/pull/1436#issuecomment-2969540413
@CTTY This sounds reasonable to me, thanks for the heads up :100: For my understanding, the linked PRs are fundamentally changing the `Transaction` usage and splitting things up? E.g. there are more well-defined actions on the new path, such as an `UpdateLocationAction`. It doesn't look like this affects the `FastAppendAction`, which is what this doc PR adds an example for. Is this code path changing soon in a follow-up, after the other PRs have been merged? -- 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
