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 b154623d9fac [MINOR][TESTS] Always remove spark.master in 
ReusedConnectTestCase
b154623d9fac is described below

commit b154623d9fac162d227e1f779b962ec37d0cbb7d
Author: Hyukjin Kwon <[email protected]>
AuthorDate: Thu Jun 27 12:30:38 2024 +0900

    [MINOR][TESTS] Always remove spark.master in ReusedConnectTestCase
    
    ### What changes were proposed in this pull request?
    
    This PR proposes to always remove `spark.master` in `ReusedConnectTestCase`.
    
    ### Why are the changes needed?
    
    More just for sanity. The tests should only run with `spark.remote`
    
    ### Does this PR introduce _any_ user-facing change?
    
    No, test-only
    
    ### How was this patch tested?
    
    Fixed unittest
    
    ### Was this patch authored or co-authored using generative AI tooling?
    
    No.
    
    Closes #47113 from HyukjinKwon/minor-test-test.
    
    Authored-by: Hyukjin Kwon <[email protected]>
    Signed-off-by: Hyukjin Kwon <[email protected]>
---
 python/pyspark/testing/connectutils.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/python/pyspark/testing/connectutils.py 
b/python/pyspark/testing/connectutils.py
index b3004693724b..2f18cd8a6ccd 100644
--- a/python/pyspark/testing/connectutils.py
+++ b/python/pyspark/testing/connectutils.py
@@ -177,6 +177,8 @@ class ReusedConnectTestCase(unittest.TestCase, 
SQLTestUtils, PySparkErrorTestUti
         conf = SparkConf(loadDefaults=False)
         # Make the server terminate reattachable streams every 1 second and 
123 bytes,
         # to make the tests exercise reattach.
+        if conf._jconf is not None:
+            conf._jconf.remove("spark.master")
         conf.set("spark.connect.execute.reattachable.senderMaxStreamDuration", 
"1s")
         conf.set("spark.connect.execute.reattachable.senderMaxStreamSize", 
"123")
         return conf


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to