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

Upgrade ScalaTest plugin to respect JAVA_HOME #2088

Merged
merged 1 commit into from
Apr 7, 2021

Conversation

gerashegalov
Copy link
Collaborator

@gerashegalov gerashegalov commented Apr 7, 2021

Signed-off-by: Gera Shegalov gera@apache.org

Spark/RAPIDS plugin contains tests requiring special settings for Java 11. E.g., by default the test
test GpuArrowColumnarBatchBuilder retains reference of ArrowBuf triggers an error with JDK11.

If we use JDK11 by default, setting JAVA_HOME to a JDK8 location does not fix the problem due to
scalatest/scalatest-maven-plugin#69. Upgrading to the ScalaTest plugin version
to consume the fix

Signed-off-by: Gera Shegalov <gera@apache.org>
@gerashegalov gerashegalov added test Only impacts tests build Related to CI / CD or cleanly building labels Apr 7, 2021
@gerashegalov gerashegalov added this to the Mar 29 - Apr 9 milestone Apr 7, 2021
@gerashegalov
Copy link
Collaborator Author

build

@gerashegalov
Copy link
Collaborator Author

mvn -Pspark300tests test -Dtests="test GpuArrowColumnarBatchBuilder retains reference of ArrowBuf" -Dscalatest-maven-plugin.version=2.0.0

✔️ mvn -Pspark300tests test -Dcuda.version=cuda11 -Dtests="test GpuArrowColumnarBatchBuilder retains reference of ArrowBuf" [ -Dscalatest-maven-plugin.version=2.0.2 ]

@jlowe jlowe merged commit 91e23b3 into NVIDIA:branch-0.5 Apr 7, 2021
@gerashegalov gerashegalov deleted the upgradeScalaTestMaven branch April 7, 2021 16:30
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
Signed-off-by: Gera Shegalov <gera@apache.org>
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
Signed-off-by: Gera Shegalov <gera@apache.org>
gerashegalov added a commit that referenced this pull request Jun 10, 2021
Tools should not unnecessarily override parent pom versions

Recurrence of #2088 

Signed-off-by: Gera Shegalov <gera@apache.org>
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 test Only impacts tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants