-
Notifications
You must be signed in to change notification settings - Fork 29k
[SPARK-38561][K8S][DOCS] Add doc for Customized Kubernetes Schedulers
#35869
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
Conversation
martin-g
left a comment
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.
I am also not a native English speaker so feel free to improve even further on my suggestions or just ignore the ones that don't feel like an improvement!
Co-authored-by: Martin Grigorov <[email protected]>
holdenk
left a comment
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
|
Merged to the current development branch :) |
dongjoon-hyun
left a comment
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, late LGTM.
|
Backport to 3.3: #35955 |
### What changes were proposed in this pull request? This is PR to doc for basic framework capability for Customized Kubernetes Schedulers. ### Why are the changes needed? Guide user how to use spark on kubernetes custom scheduler ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? CI passed Closes apache#35869 from Yikun/SPARK-38561. Authored-by: Yikun Jiang <[email protected]> Signed-off-by: Holden Karau <[email protected]>
What changes were proposed in this pull request?
This is PR to doc for basic framework capability for Customized Kubernetes Schedulers.
Why are the changes needed?
Guide user how to use spark on kubernetes custom scheduler
Does this PR introduce any user-facing change?
No
How was this patch tested?
CI passed