-
Notifications
You must be signed in to change notification settings - Fork 173
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: Improve Superset on EKS doc #530
Conversation
fix: Superset fixes for Terraform
Added new images and updated the md file for apache-super-set
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.
@sguruvar thanks for the PR! Please fix the codespell errors. Also, in future, please run pre-commit run -a
in your local workstation to avoid these kinds of errors.
@sguruvar Checkout the spelling suggestions and update the PR. |
Fixed the codespell errors |
done |
Still seeing some pre-commit errors. |
Looks like pre-commit issues are related to Spark Streaming PR. This looks good to me. |
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
What does this PR do?
馃洃 Please open an issue first to discuss any significant work and flesh out details/direction - we would hate for your time to be wasted.
Consult the CONTRIBUTING guide for submitting pull-requests.
Motivation
superset on EKS changes
More
website/docs
orwebsite/blog
section for this featurepre-commit run -a
with this PR. Link for installing pre-commit locallyFor Moderators
Additional Notes