IndexSeek commented on code in PR #1464:
URL: https://github.com/apache/iceberg-python/pull/1464#discussion_r1895072054
##########
pyiceberg/utils/config.py:
##########
@@ -84,12 +84,13 @@ def _load_yaml(directory: Optional[str]) ->
Optional[RecursiveDict]:
return file_config_lowercase
return None
- # Give priority to the PYICEBERG_HOME directory
- if pyiceberg_home_config := _load_yaml(os.environ.get(PYICEBERG_HOME)):
- return pyiceberg_home_config
- # Look into the home directory
- if pyiceberg_home_config := _load_yaml(os.path.expanduser("~")):
- return pyiceberg_home_config
+ # Directories to search for the configuration file
Review Comment:
I added a new line to show this in
https://github.com/apache/iceberg-python/pull/1464/commits/2bd69139da4585c88df73dc7f7a36903fc7daa7a,
is this what you are looking for? I just added back this comment underneath, I
hope this is what you're looking for.
##########
pyiceberg/utils/config.py:
##########
@@ -84,12 +84,13 @@ def _load_yaml(directory: Optional[str]) ->
Optional[RecursiveDict]:
return file_config_lowercase
return None
- # Give priority to the PYICEBERG_HOME directory
- if pyiceberg_home_config := _load_yaml(os.environ.get(PYICEBERG_HOME)):
- return pyiceberg_home_config
- # Look into the home directory
- if pyiceberg_home_config := _load_yaml(os.path.expanduser("~")):
- return pyiceberg_home_config
+ # Directories to search for the configuration file
Review Comment:
I added a new line to show this in
https://github.com/apache/iceberg-python/pull/1464/commits/2bd69139da4585c88df73dc7f7a36903fc7daa7a,
is this what you are looking for? I just added back this comment underneath.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
