This is an automated email from the ASF dual-hosted git repository.

kevinjqliu pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/iceberg-python.git


The following commit(s) were added to refs/heads/main by this push:
     new 7a99eb43 Avoid local Mac issues for test_bodo_nan (#2237)
7a99eb43 is described below

commit 7a99eb43e1c41d24f6d3a2a10410765f021c0e6c
Author: Ehsan Totoni <[email protected]>
AuthorDate: Tue Jul 22 13:39:41 2025 -0400

    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]>
---
 tests/integration/test_reads.py | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/tests/integration/test_reads.py b/tests/integration/test_reads.py
index 96844fd9..a33b1a36 100644
--- a/tests/integration/test_reads.py
+++ b/tests/integration/test_reads.py
@@ -342,7 +342,11 @@ def test_daft_nan_rewritten(catalog: Catalog) -> None:
 @pytest.mark.integration
 @pytest.mark.filterwarnings("ignore")
 @pytest.mark.parametrize("catalog", 
[pytest.lazy_fixture("session_catalog_hive"), 
pytest.lazy_fixture("session_catalog")])
-def test_bodo_nan(catalog: Catalog) -> None:
+def test_bodo_nan(catalog: Catalog, monkeypatch: pytest.MonkeyPatch) -> None:
+    # Avoid local Mac issues (see 
https://github.com/apache/iceberg-python/issues/2225)
+    monkeypatch.setenv("BODO_DATAFRAME_LIBRARY_RUN_PARALLEL", "0")
+    monkeypatch.setenv("FI_PROVIDER", "tcp")
+
     table_test_null_nan_rewritten = 
catalog.load_table("default.test_null_nan_rewritten")
     df = table_test_null_nan_rewritten.to_bodo()
     assert len(df) == 3

Reply via email to