[GitHub] [iceberg] Fokko commented on pull request #5488: Python: Add configuration

[GitBox <gi...@apache.org>]

Fokko commented on PR #5488:
URL: https://github.com/apache/iceberg/pull/5488#issuecomment-1217658071

   @rdblue Changed the config file loading. Also pushed two more improvements:
   - Refactored the lazy loading of the catalogs a bit. In the future when we have more catalogs, we want to be mindful to not load everything. So generalized this a bit which makes it more extensible.
   - I'veintroduce a `type` property in the config to explicitly load a catalog. This is the same as in Java `spark.sql.catalog.hive_prod.type = hive`. For the CLI we still infer this from the properties. It tries to determine the type as we did before.


-- 
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