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

CI Add unit tests running for Spark 3.0.1 #418

Merged
merged 2 commits into from
Jul 24, 2020

Conversation

tgravescs
Copy link
Collaborator

Nightly and Premerge run unit tests against Spark 3.0.1 as well. 3.1.0 have a few failures we need to fix first.

Let me know thoughts if we think this will take to long for premerge.

@tgravescs tgravescs changed the title Add unit tests running for Spark 3.0.1 CI Add unit tests running for Spark 3.0.1 Jul 23, 2020
@tgravescs
Copy link
Collaborator Author

build

Copy link
Collaborator

@revans2 revans2 left a comment

Choose a reason for hiding this comment

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

Is there any way we can make these run in parallel? I think that would be ideal.

jenkins/spark-premerge-build.sh Outdated Show resolved Hide resolved
jenkins/spark-nightly-build.sh Show resolved Hide resolved
@tgravescs
Copy link
Collaborator Author

yeah I was going to look into running parallel separate. The problem is that unit tests are going to use entire GPU , so we would need multiple GPUs to run in parallel in the same build. We can certainly leave it out of premerge right now if we think it takes to long, I don't think it will be a big deal for nightly.

@revans2
Copy link
Collaborator

revans2 commented Jul 23, 2020

I was thinking multiple jenkins jobs. One to test each platform. We could also drop the initial memory allocated for cudf. We don't run jobs that are that big for the unit tests. We probably could run multiple in parallel that way.

@tgravescs
Copy link
Collaborator Author

sounds good, I'll talk to Tim and Calvin to see what our ci nodes have for gpus

@tgravescs
Copy link
Collaborator Author

build

@tgravescs
Copy link
Collaborator Author

looks like this increased premerge build by about 5 minutes. So its taking around 42 minutes now.

@tgravescs tgravescs merged commit aadf2ec into NVIDIA:branch-0.2 Jul 24, 2020
@jlowe jlowe added the build Related to CI / CD or cleanly building label Jul 24, 2020
@jlowe jlowe added this to the Jul 20 - Jul 31 milestone Jul 24, 2020
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
* Add unit tests running for Spark 3.0.1

* dont' run python integration tests for other versions of Spark

Co-authored-by: Thomas Graves <[email protected]>
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
* Add unit tests running for Spark 3.0.1

* dont' run python integration tests for other versions of Spark

Co-authored-by: Thomas Graves <[email protected]>
pxLi pushed a commit to pxLi/spark-rapids that referenced this pull request May 12, 2022
tgravescs pushed a commit to tgravescs/spark-rapids that referenced this pull request Nov 30, 2023
[auto-merge] bot-auto-merge-branch-22.08 to branch-22.10 [skip ci] [bot]
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.

3 participants