Skip to content

[SPARK-51909][INFRA] Add a scheduled workflow for PySpark Classic-only#50703

Closed
zhengruifeng wants to merge 4 commits intoapache:masterfrom
zhengruifeng:infra_py_classic_only
Closed

[SPARK-51909][INFRA] Add a scheduled workflow for PySpark Classic-only#50703
zhengruifeng wants to merge 4 commits intoapache:masterfrom
zhengruifeng:infra_py_classic_only

Conversation

@zhengruifeng
Copy link
Copy Markdown
Contributor

@zhengruifeng zhengruifeng commented Apr 25, 2025

What changes were proposed in this pull request?

Add a scheduled (every 3 days) workflow for PySpark Classic-only

Why are the changes needed?

pyspark-classic should work without connect-related dependencies, all tests should pass

Does this PR introduce any user-facing change?

no

How was this patch tested?

PR builder with

default: '{"PYSPARK_IMAGE_TO_TEST": "python-311-classic-only", "PYTHON_TO_TEST": "python3.11"}'

https://github.com/zhengruifeng/spark/actions/runs/14640857368/job/41082669149

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

no

Copy link
Copy Markdown
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, LGTM. Thank you for adding this test coverage, @zhengruifeng .

Copy link
Copy Markdown
Contributor

@LuciferYang LuciferYang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, LGTM

@zhengruifeng
Copy link
Copy Markdown
Contributor Author

thanks you all, merged to master

@zhengruifeng zhengruifeng deleted the infra_py_classic_only branch April 25, 2025 02:38
@zhengruifeng
Copy link
Copy Markdown
Contributor Author

oops, seems the linter was broken, will fix in https://github.com/apache/spark/pull/50705/files

yhuang-db pushed a commit to yhuang-db/spark that referenced this pull request Jun 9, 2025
### What changes were proposed in this pull request?
Add a scheduled (every 3 days) workflow for PySpark Classic-only

### Why are the changes needed?
pyspark-classic should work without connect-related dependencies, all tests should pass

### Does this PR introduce _any_ user-facing change?
no

### How was this patch tested?
PR builder with
```
default: '{"PYSPARK_IMAGE_TO_TEST": "python-311-classic-only", "PYTHON_TO_TEST": "python3.11"}'
```

https://github.com/zhengruifeng/spark/actions/runs/14640857368/job/41082669149

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

Closes apache#50703 from zhengruifeng/infra_py_classic_only.

Authored-by: Ruifeng Zheng <[email protected]>
Signed-off-by: Ruifeng Zheng <[email protected]>
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.

4 participants