You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "nastra (via GitHub)" <gi...@apache.org> on 2023/06/06 07:56:11 UTC

[GitHub] [iceberg] nastra commented on pull request #7759: Core: View metadata implementation

nastra commented on PR #7759:
URL: https://github.com/apache/iceberg/pull/7759#issuecomment-1578122566

   > @nastra Is it possible to abstract a common interface for table and view because they have very similar methods?
   
   @ConeyLiu I don't think this is desirable, because `ViewMetadata` and `TableMetadata` only overlap in a few fields, so adding an abstract common interface might add unnecessary complexity 


-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org