-
-
Notifications
You must be signed in to change notification settings - Fork 4
[Merged by Bors] - Docs: new usage guide and index page #229
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
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.
Some ideas for the diagram:
- remove the ...-driver-pod-spec-conf-map since it's not created by the operator
- remove the ...-submit-job config map. It's only needed for logging of the
spark-submit
output. Rather irrelevant for a first high level diagram. - maybe use a different shapes (not just colors) for different objects and get rid ot the large ServiceAccount or ConfigMap titles.
- maybe also integrate the history server with the S3 bucket for the event files.
Co-authored-by: Razvan-Daniel Mihai <[email protected]>
Co-authored-by: Razvan-Daniel Mihai <[email protected]>
Co-authored-by: Razvan-Daniel Mihai <[email protected]>
I've cleaned up the diagram and added more info on the history server. @razvan |
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.
nice
bors merge |
# Description *Please add a description here. This will become the commit message of the merge request later.* Co-authored-by: Felix Hennig <[email protected]> Co-authored-by: Razvan-Daniel Mihai <[email protected]>
Pull request successfully merged into main. Build succeeded: |
*Please add a description here. This will become the commit message of the merge request later.* Co-authored-by: Felix Hennig <[email protected]> Co-authored-by: Razvan-Daniel Mihai <[email protected]>
Description
Please add a description here. This will become the commit message of the merge request later.
Definition of Done Checklist
Once the review is done, comment
bors r+
(orbors merge
) to merge. Further information