Re: [I] Avoid Vec allocation in Transaction::do_commit [iceberg-rust]

CTTY commented on issue #1437: URL: https://github.com/apache/iceberg-rust/issues/1437#issuecomment-2998147938 This is closed by https://github.com/apache/iceberg-rust/pull/1451 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [I] Avoid Vec allocation in Transaction::do_commit [iceberg-rust]

CTTY closed issue #1437: Avoid Vec allocation in Transaction::do_commit URL: https://github.com/apache/iceberg-rust/issues/1437 -- 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.

[I] Avoid Vec allocation in Transaction::do_commit [iceberg-rust]

CTTY opened a new issue, #1437: URL: https://github.com/apache/iceberg-rust/issues/1437 As discussed here: https://github.com/apache/iceberg-rust/pull/1433#discussion_r2141327842 We should borrow actions in the vector instead of cloning the vector along with the removal of `updates`