Skip to content

[SPARK-47757][SPARK-47756][CONNECT][PYTHON][TESTS] Make testing Spark Connect server having pyspark.core#46036

Closed
HyukjinKwon wants to merge 1 commit intoapache:masterfrom
HyukjinKwon:do-not-merge-python
Closed

[SPARK-47757][SPARK-47756][CONNECT][PYTHON][TESTS] Make testing Spark Connect server having pyspark.core#46036
HyukjinKwon wants to merge 1 commit intoapache:masterfrom
HyukjinKwon:do-not-merge-python

Conversation

@HyukjinKwon
Copy link
Member

@HyukjinKwon HyukjinKwon commented Apr 13, 2024

What changes were proposed in this pull request?

This PR proposes to testing PySpark Connect server to have pyspark.core package by running Python workers once (and they will be reused later).

Also, it fixes accumulator processing available in Python workers, that are used for Python profilers.

Why are the changes needed?

To make sure on test coverage with pyspark-connect package for UDFProfilerParityTests and MemoryProfilerParityTests.

Does this PR introduce any user-facing change?

No, test-only.

How was this patch tested?

https://github.com/HyukjinKwon/spark/actions/runs/8682010811/job/23805712575

Was this patch authored or co-authored using generative AI tooling?

No.

@HyukjinKwon HyukjinKwon marked this pull request as draft April 13, 2024 01:27
@HyukjinKwon HyukjinKwon force-pushed the do-not-merge-python branch from 4dec0c5 to 63a8b03 Compare April 15, 2024 01:24
@github-actions github-actions bot added the CORE label Apr 15, 2024
@HyukjinKwon HyukjinKwon changed the title [WIP][SPARK-47757][SPARK-47756][CONNECT][PYTHON][TESTS] Make testing Spark Connect server having pyspark.core [SPARK-47757][SPARK-47756][CONNECT][PYTHON][TESTS] Make testing Spark Connect server having pyspark.core Apr 15, 2024
@HyukjinKwon HyukjinKwon force-pushed the do-not-merge-python branch from 63a8b03 to 63fa6a2 Compare April 15, 2024 01:54
@HyukjinKwon HyukjinKwon marked this pull request as ready for review April 15, 2024 01:54
@HyukjinKwon HyukjinKwon force-pushed the do-not-merge-python branch 2 times, most recently from b444e78 to 17c34ab Compare April 15, 2024 02:05
@HyukjinKwon
Copy link
Member Author

@HyukjinKwon HyukjinKwon force-pushed the do-not-merge-python branch from 17c34ab to 3872971 Compare April 15, 2024 03:10
@HyukjinKwon
Copy link
Member Author

Merged to master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants