kevinjqliu commented on issue #1987: URL: https://github.com/apache/iceberg-python/issues/1987#issuecomment-2874433972
Thanks for reporting! `pyiceberg_core` / `pyiceberg-core` is marked optional since its only used when writing with pyarrow fs for tables with partition transforms. Typically we make these dependencies optional for specific use cases to keep the installation light. We _can_ add `pyiceberg-core` to be installed alongside `pyarrow` but some folks will only use the pyarrow option for reading and wont require pyiceberg-core Alternatively, maybe we can make the error message more clear, instead of `ModuleNotFoundError: No module named 'pyiceberg_core'`, maybe something like this https://github.com/apache/iceberg-python/blob/260ef54e3920d435ae3b2ccda090e66f9c1ac015/pyiceberg/catalog/__init__.py#L133-L134 WDYT? -- 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
