Commit 06404a5
Remove top-level import of
The [release candidate artifact build
environment](https://github.com/apache/iceberg-python/blob/a58f099aa501f6fd4345a331295d81fe0133554f/.github/workflows/pypi-build-artifacts.yml#L72-L74)
does not automatically install `pyarrow`. So when the import requires
`pyarrow`, it fails. See run
https://github.com/apache/iceberg-python/actions/runs/13464626812/job/37627644985
Import is via `conftest`
```
ImportError while loading conftest '/project/tests/conftest.py'.
/project/tests/conftest.py:52: in <module>
from pyiceberg.catalog import Catalog, load_catalog
../venv/lib/python3.9/site-packages/pyiceberg/catalog/__init__.py:51: in <module>
from pyiceberg.serializers import ToOutputFile
../venv/lib/python3.9/site-packages/pyiceberg/serializers.py:25: in <module>
from pyiceberg.table.metadata import TableMetadata, TableMetadataUtil
../venv/lib/python3.9/site-packages/pyiceberg/table/__init__.py:65: in <module>
from pyiceberg.io.pyarrow import ArrowScan, expression_to_pyarrow, schema_to_pyarrow
../venv/lib/python3.9/site-packages/pyiceberg/io/pyarrow.py:62: in <module>
import pyarrow as pa
E ModuleNotFoundError: No module named 'pyarrow'
```
This isnt caught in CI since we install all extra deps by default,
including `pyarrow`
Tested in the release candidate build action on my fork:
https://github.com/kevinjqliu/iceberg-python/actions/runs/13465085426 ✅
cc @geruh
---------
Co-authored-by: Fokko Driesprong <fokko@apache.org>pyarrow (#1703)1 parent a58f099 commit 06404a5
1 file changed
+9
-2
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
62 | 62 | | |
63 | 63 | | |
64 | 64 | | |
65 | | - | |
66 | 65 | | |
67 | 66 | | |
68 | 67 | | |
| |||
1150 | 1149 | | |
1151 | 1150 | | |
1152 | 1151 | | |
| 1152 | + | |
| 1153 | + | |
| 1154 | + | |
| 1155 | + | |
| 1156 | + | |
| 1157 | + | |
1153 | 1158 | | |
1154 | 1159 | | |
1155 | 1160 | | |
| |||
1770 | 1775 | | |
1771 | 1776 | | |
1772 | 1777 | | |
1773 | | - | |
| 1778 | + | |
1774 | 1779 | | |
1775 | 1780 | | |
1776 | 1781 | | |
| |||
1828 | 1833 | | |
1829 | 1834 | | |
1830 | 1835 | | |
| 1836 | + | |
| 1837 | + | |
1831 | 1838 | | |
1832 | 1839 | | |
1833 | 1840 | | |
| |||
0 commit comments