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

Minor change: Make Hybrid version a separate config like priviate repo #11973

Merged
merged 2 commits into from
Jan 20, 2025

Conversation

res-life
Copy link
Collaborator

@res-life res-life commented Jan 17, 2025

This is a follow-up PR for #11720,
to fix the issue: #11975

Like spark-rapids-private repo, Hybrid has its own repo.
Like spark-rapids-private, define a config spark-rapids-private.version to switch version.
Hybrid also should have a similar config. Release process require third-party jar has a version config.

Signed-off-by: Chong Gao [email protected]

@res-life res-life requested a review from a team as a code owner January 17, 2025 06:26
@res-life res-life requested a review from NvTimLiu January 17, 2025 06:26
@res-life
Copy link
Collaborator Author

build

@pxLi
Copy link
Collaborator

pxLi commented Jan 17, 2025

please also update scala213 pom file~

@pxLi pxLi added the build Related to CI / CD or cleanly building label Jan 17, 2025
@res-life
Copy link
Collaborator Author

build

@pxLi
Copy link
Collaborator

pxLi commented Jan 17, 2025

cc @NvTimLiu

@NvTimLiu NvTimLiu requested review from pxLi, NvTimLiu and a team January 17, 2025 09:40
@res-life res-life merged commit cd36fa9 into NVIDIA:branch-25.02 Jan 20, 2025
52 checks passed
@res-life res-life deleted the hybrid-version branch January 20, 2025 01:10
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.

4 participants