Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Drop spark-3.1.x support for spark-rapids #11041

Closed
wants to merge 2 commits into from

Conversation

NvTimLiu
Copy link
Collaborator

CI part for the issue: #10955

We'll drop spark-3.1.x support from branch-24.08, change the default spark version string to 3.2.0

This change needs to work together with the source code update for dropping spark-3.1.x support

Will keep monitoring CI jobs status after all the related changes get merged

CI part for the issue: NVIDIA#10955

We'll drop spark-3.1.x support from branch-24.08, change the default spark version string to 3.2.0

This change needs to work together with the source code update for dropping spark-3.1.x support

Will keep monitoring CI jobs status after all the related changes get merged

Signed-off-by: Tim Liu <[email protected]>
@NvTimLiu NvTimLiu added the build Related to CI / CD or cleanly building label Jun 11, 2024
@NvTimLiu NvTimLiu self-assigned this Jun 11, 2024
@NvTimLiu NvTimLiu marked this pull request as draft June 11, 2024 09:29
@NvTimLiu
Copy link
Collaborator Author

Draft the PR, because this change needs to work together with the source code update for dropping spark-3.1.x support

@NvTimLiu
Copy link
Collaborator Author

NvTimLiu commented Jul 3, 2024

build

@@ -32,7 +32,7 @@ CUDA_CLASSIFIER=${CUDA_CLASSIFIER:-"cuda11"}
CLASSIFIER=${CLASSIFIER:-"$CUDA_CLASSIFIER"} # default as CUDA_CLASSIFIER for compatibility
PROJECT_VER=${PROJECT_VER:-"24.08.0-SNAPSHOT"}
PROJECT_TEST_VER=${PROJECT_TEST_VER:-"24.08.0-SNAPSHOT"}
SPARK_VER=${SPARK_VER:-"3.1.1"}
SPARK_VER=${SPARK_VER:-"3.2.0"}
SPARK_VER_213=${SPARK_VER_213:-"3.3.0"}
Copy link
Collaborator

Choose a reason for hiding this comment

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

BTW do you know if anyone relies on the SPARK_VER_213 somewhere? it seems an unused one

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes seems it's unused

@pxLi
Copy link
Collaborator

pxLi commented Jul 3, 2024

can you also help remove 31x https://github.com/NVIDIA/spark-rapids/blob/branch-24.08/pom.xml#L866 from the test list? thanks

@NvTimLiu
Copy link
Collaborator Author

NvTimLiu commented Jul 9, 2024

Drop this, replaced by #11159

@NvTimLiu NvTimLiu closed this Jul 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Related to CI / CD or cleanly building
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants