liurenjie1024 commented on PR #1336: URL: https://github.com/apache/iceberg-rust/pull/1336#issuecomment-2918836508
> > The `SnapshotUpdate` is a super interface of tx actions that could produce a new snapshot, so in theory you could use any tx action that produces a new snapshot to do that. Currently, you could you `FastAppend`. > > Yeah I actually checked existing interface before I made this PR, I didn't any interface exposing the capability to set / update snapshot summary: > > https://github.com/apache/iceberg-rust/blob/3ea6f4791a2a4e15c4466cff7ccc478c08e2d3e3/crates/iceberg/src/transaction/append.rs#L40-L58 I think you find the correct way to do it in #1391 -- 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
