-
Notifications
You must be signed in to change notification settings - Fork 28.2k
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
[SPARK-33046][DOCS] Update how to build doc for Scala 2.13 with sbt #29921
Conversation
Test build #129306 has finished for PR 29921 at commit
|
Kubernetes integration test starting |
Kubernetes integration test status failure |
retest this please. |
Test build #129307 has finished for PR 29921 at commit
|
Kubernetes integration test starting |
Kubernetes integration test status success |
docs/building-spark.md
Outdated
@@ -265,15 +265,13 @@ Change the major Scala version using (e.g. 2.13): | |||
|
|||
./dev/change-scala-version.sh 2.13 | |||
|
|||
For Maven, please enable the profile (e.g. 2.13): | |||
Eenable the profile (e.g. 2.13): |
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.
@sarutak . nit. A typo: Eenable
-> Enable
?
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.
Ah, I'll fix it. Thanks.
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.
+1, LGTM (except one typo)
Test build #129321 has finished for PR 29921 at commit
|
Merged to master |
Kubernetes integration test starting |
Kubernetes integration test status success |
What changes were proposed in this pull request?
This PR fixes the description how to build Spark for Scala 2.13 with sbt.
In the current doc, how to build Spark for Scala 2.13 with sbt is described like:
But build fails with this command because scala-2.13 profile is not enabled and scala-parallel-collections is absent.
The correct command should be:
Why are the changes needed?
The build command is wrong.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
I checked that
sbt -Pspark-2.13
is correct with the following command:I also build the modified doc and checked the generated html: