Re: [PR] Log exception when FileIO import fails [iceberg-python]

2025-01-26 Thread via GitHub
kevinjqliu merged PR #1578: URL: https://github.com/apache/iceberg-python/pull/1578 -- 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...@i

Re: [PR] Log exception when FileIO import fails [iceberg-python]

2025-01-26 Thread via GitHub
rshkv commented on code in PR #1578: URL: https://github.com/apache/iceberg-python/pull/1578#discussion_r1929846312 ## tests/io/test_io.py: ## @@ -281,6 +282,11 @@ def test_import_file_io_does_not_exist() -> None: assert _import_file_io("pyiceberg.does.not.exist.FileIO", {}

Re: [PR] Log exception when FileIO import fails [iceberg-python]

2025-01-26 Thread via GitHub
kevinjqliu commented on code in PR #1578: URL: https://github.com/apache/iceberg-python/pull/1578#discussion_r1929839265 ## tests/io/test_io.py: ## @@ -281,6 +282,11 @@ def test_import_file_io_does_not_exist() -> None: assert _import_file_io("pyiceberg.does.not.exist.FileIO

Re: [PR] Log exception when FileIO import fails [iceberg-python]

2025-01-26 Thread via GitHub
rshkv commented on PR #1578: URL: https://github.com/apache/iceberg-python/pull/1578#issuecomment-2614539396 Updated the warnings for location providers and custom catalogs. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] Log exception when FileIO import fails [iceberg-python]

2025-01-26 Thread via GitHub
rshkv commented on code in PR #1578: URL: https://github.com/apache/iceberg-python/pull/1578#discussion_r1929836366 ## tests/io/test_io.py: ## @@ -281,6 +282,11 @@ def test_import_file_io_does_not_exist() -> None: assert _import_file_io("pyiceberg.does.not.exist.FileIO", {}

Re: [PR] Log exception when FileIO import fails [iceberg-python]

2025-01-26 Thread via GitHub
kevinjqliu commented on code in PR #1578: URL: https://github.com/apache/iceberg-python/pull/1578#discussion_r1929821411 ## pyiceberg/io/__init__.py: ## @@ -316,7 +316,7 @@ def _import_file_io(io_impl: str, properties: Properties) -> Optional[FileIO]: class_ = getattr(

[PR] Log exception when FileIO import fails [iceberg-python]

2025-01-26 Thread via GitHub
rshkv opened a new pull request, #1578: URL: https://github.com/apache/iceberg-python/pull/1578 Closes #1577. Log the underlying exception when a FileIO import fails. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and