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.

Sign up for GitHub

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

Jump to bottom

Update spark320.version to 3.2.0 #3806

Merged
merged 3 commits into from
Oct 13, 2021
Merged

Update spark320.version to 3.2.0 #3806

merged 3 commits into from
Oct 13, 2021

Conversation

pxLi
Copy link
Collaborator

@pxLi pxLi commented Oct 13, 2021 •
edited
Loading

Signed-off-by: Peixin Li pxli@nyu.edu

spark 3.2.0 rc finally passed vote, and libs has been deployed to maven central so update our dep version accordingly,

NOTE: official spark 320 bin is not available yet, we must wait for it before merge this, otherwise the shim version check could fail the nightly tests

@pxLi
Update spark320 version to 3.2.0
a56e98a 
Signed-off-by: Peixin Li <pxli@nyu.edu>
@pxLi pxLi added the build Related to CI / CD or cleanly building label Oct 13, 2021
@pxLi
Copy link
Collaborator Author

pxLi commented Oct 13, 2021

build

@abellina
Copy link
Collaborator

NOTE: official spark 320 bin is not available yet, we must wait for it before merge this, otherwise the shim version check could fail the nightly tests

@pxLi mind making this a draft while we wait?

@pxLi pxLi marked this pull request as draft October 13, 2021 01:47
@pxLi
Copy link
Collaborator Author

pxLi commented Oct 13, 2021

convert to draft while waiting for spark320 bin

@pxLi
Copy link
Collaborator Author

pxLi commented Oct 13, 2021

build

@pxLi pxLi marked this pull request as ready for review October 13, 2021 10:28
@pxLi
Copy link
Collaborator Author

pxLi commented Oct 13, 2021

spark320 bin is available. convert to ready for review

tgravescs
tgravescs approved these changes Oct 13, 2021
View reviewed changes
Copy link
Collaborator

@tgravescs tgravescs left a comment

Choose a reason for hiding this comment

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

we need to move spark 3.2.0 out of snapshot as well, I can do that as followup pr to this

@tgravescs tgravescs merged commit 04d208a into NVIDIA:branch-21.10 Oct 13, 2021
@pxLi
Copy link
Collaborator Author

pxLi commented Oct 14, 2021

we need to move spark 3.2.0 out of snapshot as well, I can do that as followup pr to this

thanks for taking care of the profile change

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tgravescs tgravescs tgravescs approved these changes

@abellina abellina abellina approved these changes

@GaryShen2008 GaryShen2008 Awaiting requested review from GaryShen2008 GaryShen2008 is a code owner

@jlowe jlowe Awaiting requested review from jlowe jlowe is a code owner

@NvTimLiu NvTimLiu Awaiting requested review from NvTimLiu NvTimLiu is a code owner

@revans2 revans2 Awaiting requested review from revans2 revans2 is a code owner

Assignees
No one assigned
Labels
build Related to CI / CD or cleanly building
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@pxLi @abellina @tgravescs