Skip to content

Commit 7a99eb4

Browse files
ehsantnkevinjqliu
andauthored
Avoid local Mac issues for test_bodo_nan (#2237)
<!-- Thanks for opening a pull request! --> <!-- In the case this PR will resolve an issue, please replace ${GITHUB_ISSUE_ID} below with the actual Github issue id. --> Closes #2225. # Rationale for this change Some Mac laptops have MPI initialization issues that this fixes. # Are these changes tested? Tested on a Mac laptop that used to fail. # Are there any user-facing changes? <!-- In the case of user-facing changes, please add the changelog label. --> No. --------- Co-authored-by: Kevin Liu <[email protected]>
1 parent b15937a commit 7a99eb4

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

tests/integration/test_reads.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -342,7 +342,11 @@ def test_daft_nan_rewritten(catalog: Catalog) -> None:
342342
@pytest.mark.integration
343343
@pytest.mark.filterwarnings("ignore")
344344
@pytest.mark.parametrize("catalog", [pytest.lazy_fixture("session_catalog_hive"), pytest.lazy_fixture("session_catalog")])
345-
def test_bodo_nan(catalog: Catalog) -> None:
345+
def test_bodo_nan(catalog: Catalog, monkeypatch: pytest.MonkeyPatch) -> None:
346+
# Avoid local Mac issues (see https://github.com/apache/iceberg-python/issues/2225)
347+
monkeypatch.setenv("BODO_DATAFRAME_LIBRARY_RUN_PARALLEL", "0")
348+
monkeypatch.setenv("FI_PROVIDER", "tcp")
349+
346350
table_test_null_nan_rewritten = catalog.load_table("default.test_null_nan_rewritten")
347351
df = table_test_null_nan_rewritten.to_bodo()
348352
assert len(df) == 3

0 commit comments

Comments
 (0)