kevinjqliu commented on PR #1140: URL: https://github.com/apache/iceberg-python/pull/1140#issuecomment-2442147026
> I'm positive about this change. The only consideration I could make is that we hide the SqlCatalog behind the InMemoryCatalog. Maybe it is interesting for folks to know that they can easily switch to a persistent catalog. What are your thoughts? I think it would be good to document the InMemoryCatalog, perhaps in the [catalog section of the configuration page](https://py.iceberg.apache.org/configuration/#catalogs). We can mention that it uses the SqlCatalog under the hood and to use another catalog implementation to persist the catalog 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
