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 d72efc038124 [SPARK-46687][TESTS][PYTHON][FOLLOW-UP] Skip 
MemoryProfilerParityTests when codecov enabled
d72efc038124 is described below

commit d72efc0381246370d3efbcd045637dd85ebfcd8f
Author: Hyukjin Kwon <[email protected]>
AuthorDate: Thu Feb 15 14:49:36 2024 +0900

    [SPARK-46687][TESTS][PYTHON][FOLLOW-UP] Skip MemoryProfilerParityTests when 
codecov enabled
    
    ### What changes were proposed in this pull request?
    
    This is a followup of https://github.com/apache/spark/pull/44775 that skips 
the tests with codecov on. It fails now 
(https://github.com/apache/spark/actions/runs/7709423681/job/21010676103) and 
the coverage report is broken.
    
    ### Why are the changes needed?
    
    To recover the test coverage report.
    
    ### Does this PR introduce _any_ user-facing change?
    
    No, test-only.
    
    ### How was this patch tested?
    
    Manually tested.
    
    ### Was this patch authored or co-authored using generative AI tooling?
    
    No.
    
    Closes #45112 from HyukjinKwon/SPARK-46687-followup.
    
    Authored-by: Hyukjin Kwon <[email protected]>
    Signed-off-by: Hyukjin Kwon <[email protected]>
---
 python/pyspark/tests/test_memory_profiler.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/python/pyspark/tests/test_memory_profiler.py 
b/python/pyspark/tests/test_memory_profiler.py
index 3af35a7b43ca..ac3dc34d3474 100644
--- a/python/pyspark/tests/test_memory_profiler.py
+++ b/python/pyspark/tests/test_memory_profiler.py
@@ -203,6 +203,9 @@ class MemoryProfilerTests(PySparkTestCase):
         df.mapInPandas(map, schema=df.schema).collect()
 
 
[email protected](
+    "COVERAGE_PROCESS_START" in os.environ, "Fails with coverage enabled, 
skipping for now."
+)
 @unittest.skipIf(not has_memory_profiler, "Must have memory-profiler 
installed.")
 class MemoryProfiler2TestsMixin:
     @contextmanager


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

Reply via email to