(spark) branch master updated: [MINOR][PYTHON][TESTS] Retry `test_map_in_pandas_with_column_vector`

[gu...@apache.org]

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

gurwls223 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/master by this push:
     new d2033282ecb [MINOR][PYTHON][TESTS] Retry `test_map_in_pandas_with_column_vector`
d2033282ecb is described below

commit d2033282ecb25d3ecc41eeee3205183b204a75a1
Author: Ruifeng Zheng <ru...@apache.org>
AuthorDate: Wed Jan 10 09:38:54 2024 +0900

    [MINOR][PYTHON][TESTS] Retry `test_map_in_pandas_with_column_vector`
    
    ### What changes were proposed in this pull request?
    Retry `test_map_in_pandas_with_column_vector` and its parity test
    
    ### Why are the changes needed?
    I am seeing this test and its parity test failing from time to time, and then fails `pyspark-sql` and `pyspark-connect`.
    
    It seems due to some log4j issue, e.g
    
    https://github.com/zhengruifeng/spark/actions/runs/7459243602/job/20294868487
    ```
    test_map_in_pandas_with_column_vector (pyspark.sql.tests.pandas.test_pandas_map.MapInPandasTests) ... ERROR StatusConsoleListener An exception occurred processing Appender File
     java.lang.IllegalArgumentException: found 1 argument placeholders, but provided 0 for pattern `0, VisitedIndex{visitedIndexes={}}: [] r:0`
            at org.apache.logging.log4j.message.ParameterFormatter.formatMessage(ParameterFormatter.java:233)
    ```
    
    https://github.com/apache/spark/actions/runs/7460093200/job/20297508703
    ```
      test_map_in_pandas_with_column_vector (pyspark.sql.tests.connect.test_parity_pandas_map.MapInPandasParityTests) ... ERROR StatusConsoleListener An exception occurred processing Appender File
     java.lang.IllegalArgumentException: found 1 argument placeholders, but provided 0 for pattern `0, VisitedIndex{visitedIndexes={}}: [] r:0`
            at org.apache.logging.log4j.message.ParameterFormatter.formatMessage(ParameterFormatter.java:233)
            at org.apache.logging.log4j.message.ParameterizedMessage.formatTo(ParameterizedMessage.java:266)
            at
    ```
    
    this PR simply attempt to retry it after failures
    
    ### Does this PR introduce _any_ user-facing change?
    no, test-only
    
    ### How was this patch tested?
    ci
    
    ### Was this patch authored or co-authored using generative AI tooling?
    no
    
    Closes #44641 from zhengruifeng/py_test_retry_mip.
    
    Authored-by: Ruifeng Zheng <ru...@apache.org>
    Signed-off-by: Hyukjin Kwon <gu...@apache.org>
---
 python/pyspark/sql/tests/pandas/test_pandas_map.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/python/pyspark/sql/tests/pandas/test_pandas_map.py b/python/pyspark/sql/tests/pandas/test_pandas_map.py
index ec9f208d08f..8f7229e1d74 100644
--- a/python/pyspark/sql/tests/pandas/test_pandas_map.py
+++ b/python/pyspark/sql/tests/pandas/test_pandas_map.py
@@ -31,7 +31,7 @@ from pyspark.testing.sqlutils import (
     pandas_requirement_message,
     pyarrow_requirement_message,
 )
-from pyspark.testing.utils import QuietTest
+from pyspark.testing.utils import QuietTest, eventually
 
 if have_pandas:
     import pandas as pd
@@ -381,6 +381,7 @@ class MapInPandasTestsMixin:
         self.assertEqual(sorted(actual), sorted(expected))
 
     # SPARK-33277
+    @eventually(timeout=180, catch_assertions=True)
     def test_map_in_pandas_with_column_vector(self):
         path = tempfile.mkdtemp()
         shutil.rmtree(path)


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org