pvary commented on issue #9514: URL: https://github.com/apache/iceberg/issues/9514#issuecomment-1899924625
> @nk1506 which particular methods did you have in mind that would make sense to extract into `IcebergMetadata`? @nastra: When managing tables and views in the catalogs we often have to handle them similarly. Especially when we drop, alter a table but even when creating them we need to manage the metadata similarly. Also, IMHO it is important to have a lock around the `commit` operations for the views as well, and it is a quite complicated logic which is easier to abstract out when the view and table metadata has a common interface. Also this commonality is already highlighted by having the same names for the methods. It is just not formalized. -- 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
