DerGut commented on issue #700: URL: https://github.com/apache/iceberg-rust/issues/700#issuecomment-2848180015
Hi 👋 wanted to mention that I started to look into implementing the [`Catalog::update_table`](https://github.com/apache/iceberg-rust/blob/b4bc6dd15ae4f7a4b1c9ce555e3458089a83e228/crates/iceberg/src/catalog/mod.rs#L96) trait method for the remaining catalogs which IIUC builds the first part of the commit path mentioned above. I mostly looked at the [`MemoryCatalog`](https://github.com/apache/iceberg-rust/blob/b4bc6dd15ae4f7a4b1c9ce555e3458089a83e228/crates/catalog/memory/src/catalog.rs#L59) for now, but already found some abstractions that could be reused for other catalogs as well, so I might just continue to work on them. Please share if there are things we need to sort out first. I can also create a dedicated ticket for more in-depth discussions if preferred! :) -- 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
