Conversation
3a6dded to
76b6d7c
Compare
76b6d7c to
3c61314
Compare
|
This PR is stale because it has been open 20 days with no activity. Remove stale label or comment or this will be closed in 10 days. |
3c61314 to
14718a7
Compare
|
@pan3793, I have bumped version to 4.1.1 and excluded netty dependencies from spark-sql dependency. PTAL. |
|
@SteNicholas AFAIK, Spark 4.1 uses Netty 4.2 exclusive classes, so integration test should fail (suppose you also exclude Netty 4.2 from test scope), in this case, I'm afraid the right path is - upgrading Netty 4.2 for Celeborn first, then try integration with Spark 4.1 |
|
This PR is stale because it has been open 20 days with no activity. Remove stale label or comment or this will be closed in 10 days. |
14718a7 to
4122cd1
Compare
|
seems you forgot to update the CI build and test workflow |
4122cd1 to
10df18b
Compare
79df55d to
028fcc0
Compare
028fcc0 to
876b80e
Compare
|
@pan3793, sorry for missing CI of |
|
Merged to main(v0.7.0). |
What changes were proposed in this pull request?
Support Spark 4.1.
Why are the changes needed?
Spark 4.1.1 has already released, which release notes refer to Spark Release 4.1.1.
Does this PR resolve a correctness bug?
No.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
CI.