danielcweeks commented on PR #12014: URL: https://github.com/apache/iceberg/pull/12014#issuecomment-2637830139
I feel like structurally we're not approaching this the right way. We already have a `BaseMetadataTable` implementation used for metadata tables. View metadata is a table and we should reuse that concept as opposed to creating a new table-like abstraction. Right now that implementation has ties to a table but we might be able to leverage that for views by using `ViewHistoryTable extends BaseMetadataTable` and wrapping the view information in a table-like structure as opposed to creating a new/parallel structure for view metadata. -- 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
