-
Notifications
You must be signed in to change notification settings - Fork 28.1k
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-31960][YARN][DOCS][FOLLOW-UP] Document the behaviour change of Hadoop's classpath propagation in migration guide #30903
Conversation
cc @dbtsai, @viirya, @tgravescs, @dongjoon-hyun. could you take a look when you find some time? |
88a52d9
to
1bc2b89
Compare
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.
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.
lgtm
1bc2b89
to
aa49b43
Compare
Test build #133281 has finished for PR 30903 at commit
|
Let me just merge this in. The last commit was removing a space ... |
Merged to master and branch-3.1. |
… Hadoop's classpath propagation in migration guide ### What changes were proposed in this pull request? This PR is a followup of #28788, and proposes to update migration guide. ### Why are the changes needed? To tell users about the behaviour change. ### Does this PR introduce _any_ user-facing change? Yes, it updates migration guides for users. ### How was this patch tested? GitHub Actions' documentation build should test it. Closes #30903 from HyukjinKwon/SPARK-31960-followup. Authored-by: HyukjinKwon <gurwls223@apache.org> Signed-off-by: HyukjinKwon <gurwls223@apache.org> (cherry picked from commit d98c216) Signed-off-by: HyukjinKwon <gurwls223@apache.org>
Test build #133285 has finished for PR 30903 at commit
|
Kubernetes integration test starting |
Kubernetes integration test starting |
Kubernetes integration test status failure |
Kubernetes integration test status failure |
Sorry for the late reply. Thank you @HyukjinKwon to help on this documentation. |
What changes were proposed in this pull request?
This PR is a followup of #28788, and proposes to update migration guide.
Why are the changes needed?
To tell users about the behaviour change.
Does this PR introduce any user-facing change?
Yes, it updates migration guides for users.
How was this patch tested?
GitHub Actions' documentation build should test it.