syun64 commented on issue #938: URL: https://github.com/apache/iceberg-python/issues/938#issuecomment-2234935944
Hi @jackye1995 - I'm of the opinion that this is totally fine. If the client wants to have their own catalog implementation for any of many possible reasons (e.g. because it requires a specific auth mechanism) they should be allowed to. I'm actually wondering what about the state of code right now, that would prevent them from having their own implementation. I want to hold off on giving a definite answer before actually trying this out, but I think even now, they should be able to implement the `Catalog` or `MetastoreCatalog` classes and use their own Catalog. In that case, maybe adding a bit more documentation on the topic to explain how they can go about implementing their own `Catalog`s will be helpful to the users. -- 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