-
Notifications
You must be signed in to change notification settings - Fork 242
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
Smoke test with '--package' to fetch the plugin jar #10238
Conversation
To fix NVIDIA#10160 Run SPARK_SHELL_SMOKE_TEST with '--package' to fetch the plugin jar Can downloading the plugin jar from specified artifact repo by '--repositories' This test can fetch plugin jar from internal maven repo, maven central, or Sonatype staging repo. Signed-off-by: Tim Liu <[email protected]>
This suggestion is reasonable Co-authored-by: Jason Lowe <[email protected]>
Signed-off-by: Tim Liu <[email protected]>
build |
CI failed due to |
build |
@@ -304,6 +304,11 @@ if [[ $TEST_MODE == "DEFAULT" ]]; then | |||
PYSP_TEST_spark_shuffle_manager=com.nvidia.spark.rapids.${SHUFFLE_SPARK_SHIM}.RapidsShuffleManager \ | |||
./run_pyspark_from_build.sh | |||
|
|||
SPARK_SHELL_SMOKE_TEST=1 \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a limitation in Spark that spark currently does not support classifiers with --packages
, https://issues.apache.org/jira/browse/SPARK-20075
It only works on the default cuda11 jar, Can not work on cuda12 and arm64 classifiers
all tests should eventually disable UI #10227 |
To fix #10160
Run SPARK_SHELL_SMOKE_TEST with '--package' to fetch the plugin jar
Can downloading the plugin jar from specified artifact repo by '--repositories'
This test can fetch plugin jar from internal maven repo, maven central, or Sonatype staging repo.
Signed-off-by: Tim Liu [email protected]