bdilday opened a new pull request, #45: URL: https://github.com/apache/iceberg-python/pull/45
I found that I was unable to change a catalog URI after it had been configured by an environment variable. Example, ``` import os os.environ["PYICEBERG_CATALOG__SOMEDB__URI"] = "https://service-one.io"; from pyiceberg.catalog import load_catalog catalog = load_catalog("somedb", uri="https://service-two.io";) ``` [tries to connect to `https://service-one.io`] I think the more intuitive behavior is that the URI would be initialized based on the environment variable (if present) but that the argument passed to `load_catalog` would be able to override the value. This PR updates the behavior so that the passed in argument takes precedence, by swapping the lhs and rhs preference in the `merge_config` method. -- 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
