Skip to content

Conversation

@xianzhe-databricks
Copy link
Contributor

@xianzhe-databricks xianzhe-databricks commented Nov 26, 2025

What changes were proposed in this pull request?

Why are the changes needed?

To regain test coverage

Does this PR introduce any user-facing change?

no

How was this patch tested?

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

no

@xianzhe-databricks xianzhe-databricks changed the title [SPARK-54481][WIP][TEST][PYTHON][CONNECT]skip skipping [SPARK-54481][SPARK-54495][SPARK-54483][WIP][TEST][PYTHON][CONNECT]skip skipping Nov 26, 2025
@xianzhe-databricks xianzhe-databricks deleted the enable-tests branch November 27, 2025 14:41
@xianzhe-databricks
Copy link
Contributor Author

#53250

@xianzhe-databricks xianzhe-databricks restored the enable-tests branch November 28, 2025 10:53
@xianzhe-databricks xianzhe-databricks changed the title [SPARK-54481][SPARK-54495][SPARK-54483][WIP][TEST][PYTHON][CONNECT]skip skipping [SPARK-54481][SPARK-54495][SPARK-54483][WIP][TEST][PYTHON][CONNECT]Reenable a few spark connect tests Nov 28, 2025
@HyukjinKwon HyukjinKwon changed the title [SPARK-54481][SPARK-54495][SPARK-54483][WIP][TEST][PYTHON][CONNECT]Reenable a few spark connect tests [SPARK-54481][SPARK-54495][SPARK-54483][TESTS][PYTHON][CONNECT]Reenable a few spark connect tests Dec 1, 2025
@HyukjinKwon HyukjinKwon changed the title [SPARK-54481][SPARK-54495][SPARK-54483][TESTS][PYTHON][CONNECT]Reenable a few spark connect tests [SPARK-54481][SPARK-54495][SPARK-54483][TESTS][PYTHON][CONNECT] Reenable a few spark connect tests Dec 1, 2025
@HyukjinKwon
Copy link
Member

Merged to master and branch-4.1.

This is a test-only so won't affect the release.

@HyukjinKwon
Copy link
Member

cc @dongjoon-hyun FYI

HyukjinKwon pushed a commit that referenced this pull request Dec 1, 2025
…ble a few spark connect tests

### What changes were proposed in this pull request?

* This PR re-enables 3 tests, which only fail during the cross-version scenario: Spark Connect 4.0 client <-> 4.2 Spark Server
* We fixed these tests on the 4.0 branch separately in another PR #53250
* ‼️ This PR needs to be backported to `branch-4.1`. How can I make it possible? HyukjinKwon

### Why are the changes needed?
To regain test coverage

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

no

### How was this patch tested?

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

no

Closes #53235 from xianzhe-databricks/enable-tests.

Authored-by: xianzhe-databricks <[email protected]>
Signed-off-by: Hyukjin Kwon <[email protected]>
(cherry picked from commit 54cde81)
Signed-off-by: Hyukjin Kwon <[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.

2 participants