-
Notifications
You must be signed in to change notification settings - Fork 28.5k
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-36384][CORE][DOC] Add doc for shuffle checksum #33637
Conversation
Kubernetes integration test starting |
Kubernetes integration test status success |
Test build #142038 has finished for PR 33637 at commit
|
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.
Looks good to me.
### What changes were proposed in this pull request? Add doc for the shuffle checksum configs in `configuration.md`. ### Why are the changes needed? doc ### Does this PR introduce _any_ user-facing change? No, since Spark 3.2 hasn't been released. ### How was this patch tested? Pass existed tests. Closes #33637 from Ngone51/SPARK-36384. Authored-by: yi.wu <yi.wu@databricks.com> Signed-off-by: Hyukjin Kwon <gurwls223@apache.org> (cherry picked from commit 3b92c72) Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
Merged to master and branch-3.2. |
Late LGTM, thanks for adding this @Ngone51 ! |
Thanks all! |
1 similar comment
Thanks all! |
What changes were proposed in this pull request?
Add doc for the shuffle checksum configs in
configuration.md
.Why are the changes needed?
doc
Does this PR introduce any user-facing change?
No, since Spark 3.2 hasn't been released.
How was this patch tested?
Pass existed tests.