Skip to content

Conversation

@pull
Copy link

@pull pull bot commented Dec 1, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

HeartSaVioR and others added 2 commits December 1, 2025 17:00
…ing tests for connect compat test CI

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

This PR proposes to re-enable streaming tests for connect compatibility test CI.

### Why are the changes needed?

They were disabled due to failure, but I can't reproduce these failures in both local and CI after installing zstandard.

Code change to trigger compatibility test CI against test branch + install zstandard:
master...HeartSaVioR:spark:WIP-investigate-ss-spark-connect-compat-test-failures-master-and-4.0

Code change to re-enable these tests during reproducing:
branch-4.0...HeartSaVioR:spark:branch-4.0-SC-213385

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

No.

### How was this patch tested?

GA run with the above reproducer setup:
https://github.com/HeartSaVioR/spark/actions/runs/19807973545/job/56745231698

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

No.

Closes #53266 from HeartSaVioR/reenable-streaming-connect-tests.

Authored-by: Jungtaek Lim <kabhwan.opensource@gmail.com>
Signed-off-by: Jungtaek Lim <kabhwan.opensource@gmail.com>
…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 <xianzhe.ma@databricks.com>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
@pull pull bot locked and limited conversation to collaborators Dec 1, 2025
@pull pull bot added the ⤵️ pull label Dec 1, 2025
@pull pull bot merged commit 54cde81 into huangxiaopingRD:master Dec 1, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants